Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move widgets into widget-specific folders #1585

Merged
merged 38 commits into from
Sep 4, 2024
Merged

Move widgets into widget-specific folders #1585

merged 38 commits into from
Sep 4, 2024

Conversation

handeyeco
Copy link
Contributor

@handeyeco handeyeco commented Sep 4, 2024

Summary:

Before:

- widgets
  - __tests__
    widget1.test.tsx
    widget2.test.tsx
  - __testdata__
    widget1.testdata.tsx
    widget2.testdata.tsx
  - __stories__
    widget1.stories.tsx
    widget2.stories.tsx
  widget1.tsx
  widget2.tsx

After:

- widgets
  - widget1
    index.ts
    widget1.tsx
    widget1.testdata.tsx
    widget1.test.tsx
    widget1.stories.tsx
  - widget2
    index.ts
    widget2.tsx
    widget2.testdata.tsx
    widget2.test.tsx
    widget2.stories.tsx

See: https://khanacademy.slack.com/archives/C01AZ9H8TTQ/p1725381544129369

Test plan:

Nothing should functionally change, just moving stuff

@handeyeco handeyeco self-assigned this Sep 4, 2024
@handeyeco handeyeco changed the title move categorizer WIP: move widgets into widget-specific folders Sep 4, 2024
Copy link
Contributor

github-actions bot commented Sep 4, 2024

Size Change: 0 B

Total Size: 861 kB

ℹ️ View Unchanged
Filename Size
packages/kas/dist/es/index.js 38.3 kB
packages/keypad-context/dist/es/index.js 760 B
packages/kmath/dist/es/index.js 4.27 kB
packages/math-input/dist/es/index.js 78 kB
packages/math-input/dist/es/strings.js 1.79 kB
packages/perseus-core/dist/es/index.js 1.48 kB
packages/perseus-editor/dist/es/index.js 277 kB
packages/perseus-linter/dist/es/index.js 22.2 kB
packages/perseus/dist/es/index.js 418 kB
packages/perseus/dist/es/strings.js 3.36 kB
packages/pure-markdown/dist/es/index.js 3.66 kB
packages/simple-markdown/dist/es/index.js 12.4 kB

compressed-size-action

@handeyeco handeyeco marked this pull request as ready for review September 4, 2024 18:44
@khan-actions-bot
Copy link
Contributor

khan-actions-bot commented Sep 4, 2024

Gerald

Required Reviewers
  • @Khan/perseus for changes to .changeset/light-poems-joke.md, dev/gallery.tsx, packages/perseus/src/__stories__/article-renderer.stories.tsx, packages/perseus/src/__testdata__/renderer.testdata.ts, packages/perseus/src/__tests__/server-item-renderer.test.tsx, packages/perseus-editor/src/__stories__/content-preview.stories.tsx, packages/perseus/src/widgets/__shared__/test-keypad-context-wrapper.tsx, packages/perseus/src/widgets/__stories__/random-widgets.stories.tsx, packages/perseus/src/widgets/__tests__/interactive-graph.test.tsx, packages/perseus/src/widgets/categorizer/categorizer.stories.tsx, packages/perseus/src/widgets/categorizer/categorizer.test.ts, packages/perseus/src/widgets/categorizer/categorizer.testdata.ts, packages/perseus/src/widgets/categorizer/categorizer.tsx, packages/perseus/src/widgets/categorizer/index.ts, packages/perseus/src/widgets/cs-program/cs-program.stories.tsx, packages/perseus/src/widgets/cs-program/cs-program.test.ts, packages/perseus/src/widgets/cs-program/cs-program.testdata.ts, packages/perseus/src/widgets/cs-program/cs-program.tsx, packages/perseus/src/widgets/cs-program/index.ts, packages/perseus/src/widgets/definition/definition.stories.tsx, packages/perseus/src/widgets/definition/definition.test.ts, packages/perseus/src/widgets/definition/definition.tsx, packages/perseus/src/widgets/definition/index.ts, packages/perseus/src/widgets/dropdown/dropdown.stories.tsx, packages/perseus/src/widgets/dropdown/dropdown.test.ts, packages/perseus/src/widgets/dropdown/dropdown.testdata.ts, packages/perseus/src/widgets/dropdown/dropdown.tsx, packages/perseus/src/widgets/dropdown/index.ts, packages/perseus/src/widgets/explanation/explanation.cypress.ts, packages/perseus/src/widgets/explanation/explanation.stories.tsx, packages/perseus/src/widgets/explanation/explanation.test.ts, packages/perseus/src/widgets/explanation/explanation.testdata.ts, packages/perseus/src/widgets/explanation/explanation.tsx, packages/perseus/src/widgets/explanation/index.ts, packages/perseus/src/widgets/expression/expression-mobile.test.tsx, packages/perseus/src/widgets/expression/expression.stories.tsx, packages/perseus/src/widgets/expression/expression.test.tsx, packages/perseus/src/widgets/expression/expression.testdata.ts, packages/perseus/src/widgets/expression/expression.tsx, packages/perseus/src/widgets/expression/index.ts, packages/perseus/src/widgets/graded-group/graded-group-answer-bar.tsx, packages/perseus/src/widgets/graded-group/graded-group.stories.tsx, packages/perseus/src/widgets/graded-group/graded-group.test.ts, packages/perseus/src/widgets/graded-group/graded-group.testdata.ts, packages/perseus/src/widgets/graded-group/graded-group.tsx, packages/perseus/src/widgets/graded-group/index.ts, packages/perseus/src/widgets/graded-group-set/graded-group-set-jipt.test.ts, packages/perseus/src/widgets/graded-group-set/graded-group-set.stories.tsx, packages/perseus/src/widgets/graded-group-set/graded-group-set.test.ts, packages/perseus/src/widgets/graded-group-set/graded-group-set.testdata.ts, packages/perseus/src/widgets/graded-group-set/graded-group-set.tsx, packages/perseus/src/widgets/graded-group-set/index.ts, packages/perseus/src/widgets/grapher/grapher.cypress.ts, packages/perseus/src/widgets/grapher/grapher.stories.tsx, packages/perseus/src/widgets/grapher/grapher.test.ts, packages/perseus/src/widgets/grapher/grapher.testdata.ts, packages/perseus/src/widgets/grapher/grapher.tsx, packages/perseus/src/widgets/grapher/index.ts, packages/perseus/src/widgets/group/group.stories.tsx, packages/perseus/src/widgets/group/group.test.tsx, packages/perseus/src/widgets/group/group.testdata.ts, packages/perseus/src/widgets/group/group.tsx, packages/perseus/src/widgets/group/index.ts, packages/perseus/src/widgets/iframe/iframe.stories.tsx, packages/perseus/src/widgets/iframe/iframe.test.ts, packages/perseus/src/widgets/iframe/iframe.testdata.ts, packages/perseus/src/widgets/iframe/iframe.tsx, packages/perseus/src/widgets/iframe/index.ts, packages/perseus/src/widgets/image/image.stories.tsx, packages/perseus/src/widgets/image/image.test.ts, packages/perseus/src/widgets/image/image.testdata.ts, packages/perseus/src/widgets/image/image.tsx, packages/perseus/src/widgets/image/index.ts, packages/perseus/src/widgets/input-number/index.ts, packages/perseus/src/widgets/input-number/input-number.stories.tsx, packages/perseus/src/widgets/input-number/input-number.test.ts, packages/perseus/src/widgets/input-number/input-number.testdata.ts, packages/perseus/src/widgets/input-number/input-number.tsx, packages/perseus/src/widgets/interaction/index.ts, packages/perseus/src/widgets/interaction/interaction.stories.tsx, packages/perseus/src/widgets/interaction/interaction.test.ts, packages/perseus/src/widgets/interaction/interaction.testdata.ts, packages/perseus/src/widgets/interaction/interaction.tsx, packages/perseus/src/widgets/label-image/index.ts, packages/perseus/src/widgets/label-image/label-image.tsx, packages/perseus/src/widgets/matcher/index.ts, packages/perseus/src/widgets/matcher/matcher.stories.tsx, packages/perseus/src/widgets/matcher/matcher.test.tsx, packages/perseus/src/widgets/matcher/matcher.testdata.ts, packages/perseus/src/widgets/matcher/matcher.tsx, packages/perseus/src/widgets/matrix/index.ts, packages/perseus/src/widgets/matrix/matrix.stories.tsx, packages/perseus/src/widgets/matrix/matrix.test.ts, packages/perseus/src/widgets/matrix/matrix.testdata.ts, packages/perseus/src/widgets/matrix/matrix.tsx, packages/perseus/src/widgets/measurer/index.ts, packages/perseus/src/widgets/measurer/measurer.tsx, packages/perseus/src/widgets/molecule/index.ts, packages/perseus/src/widgets/molecule/molecule.tsx, packages/perseus/src/widgets/number-line/index.ts, packages/perseus/src/widgets/number-line/number-line.stories.tsx, packages/perseus/src/widgets/number-line/number-line.test.ts, packages/perseus/src/widgets/number-line/number-line.testdata.ts, packages/perseus/src/widgets/number-line/number-line.tsx, packages/perseus/src/widgets/numeric-input/index.ts, packages/perseus/src/widgets/numeric-input/numeric-input.stories.tsx, packages/perseus/src/widgets/numeric-input/numeric-input.test.ts, packages/perseus/src/widgets/numeric-input/numeric-input.testdata.ts, packages/perseus/src/widgets/numeric-input/numeric-input.tsx, packages/perseus/src/widgets/orderer/index.ts, packages/perseus/src/widgets/orderer/orderer.stories.tsx, packages/perseus/src/widgets/orderer/orderer.test.ts, packages/perseus/src/widgets/orderer/orderer.testdata.ts, packages/perseus/src/widgets/orderer/orderer.tsx, packages/perseus/src/widgets/passage/index.ts, packages/perseus/src/widgets/passage/passage.stories.tsx, packages/perseus/src/widgets/passage/passage.tsx, packages/perseus/src/widgets/passage-ref/index.ts, packages/perseus/src/widgets/passage-ref/passage-ref.stories.tsx, packages/perseus/src/widgets/passage-ref/passage-ref.test.ts, packages/perseus/src/widgets/passage-ref/passage-ref.testdata.ts, packages/perseus/src/widgets/passage-ref/passage-ref.tsx, packages/perseus/src/widgets/passage-ref-target/index.ts, packages/perseus/src/widgets/passage-ref-target/passage-ref-target.tsx, packages/perseus/src/widgets/phet-simulation/index.ts, packages/perseus/src/widgets/phet-simulation/phet-simulation.stories.tsx, packages/perseus/src/widgets/phet-simulation/phet-simulation.test.ts, packages/perseus/src/widgets/phet-simulation/phet-simulation.testdata.ts, packages/perseus/src/widgets/phet-simulation/phet-simulation.tsx, packages/perseus/src/widgets/plotter/index.ts, packages/perseus/src/widgets/plotter/plotter.stories.tsx, packages/perseus/src/widgets/plotter/plotter.test.tsx, packages/perseus/src/widgets/plotter/plotter.testdata.ts, packages/perseus/src/widgets/plotter/plotter.tsx, packages/perseus/src/widgets/python-program/index.ts, packages/perseus/src/widgets/python-program/python-program.stories.tsx, packages/perseus/src/widgets/python-program/python-program.test.ts, packages/perseus/src/widgets/python-program/python-program.testdata.ts, packages/perseus/src/widgets/python-program/python-program.tsx, packages/perseus/src/widgets/radio/index.ts, packages/perseus/src/widgets/radio/radio-component.tsx, packages/perseus/src/widgets/radio/radio.ts, packages/perseus/src/widgets/sorter/index.ts, packages/perseus/src/widgets/sorter/sorter.stories.tsx, packages/perseus/src/widgets/sorter/sorter.testdata.ts, packages/perseus/src/widgets/sorter/sorter.tsx, packages/perseus/src/widgets/table/index.ts, packages/perseus/src/widgets/table/table.tsx, packages/perseus/src/widgets/video/index.ts, packages/perseus/src/widgets/video/video-transcript-link.stories.tsx, packages/perseus/src/widgets/video/video-transcript-link.test.tsx, packages/perseus/src/widgets/video/video-transcript-link.tsx, packages/perseus/src/widgets/video/video.stories.tsx, packages/perseus/src/widgets/video/video.test.ts, packages/perseus/src/widgets/video/video.testdata.ts, packages/perseus/src/widgets/video/video.tsx, packages/perseus/src/widgets/categorizer/__snapshots__/categorizer.test.ts.snap, packages/perseus/src/widgets/cs-program/__snapshots__/cs-program.test.ts.snap, packages/perseus/src/widgets/definition/__snapshots__/definition.test.ts.snap, packages/perseus/src/widgets/dropdown/__snapshots__/dropdown.test.ts.snap, packages/perseus/src/widgets/explanation/__snapshots__/explanation.test.ts.snap, packages/perseus/src/widgets/graded-group/__snapshots__/graded-group.test.ts.snap, packages/perseus/src/widgets/graded-group-set/__snapshots__/graded-group-set-jipt.test.ts.snap, packages/perseus/src/widgets/graded-group-set/__snapshots__/graded-group-set.test.ts.snap, packages/perseus/src/widgets/grapher/__snapshots__/grapher.test.ts.snap, packages/perseus/src/widgets/group/__snapshots__/group.test.tsx.snap, packages/perseus/src/widgets/iframe/__snapshots__/iframe.test.ts.snap, packages/perseus/src/widgets/image/__snapshots__/image.test.ts.snap, packages/perseus/src/widgets/input-number/__snapshots__/input-number.test.ts.snap, packages/perseus/src/widgets/interaction/__snapshots__/interaction.test.ts.snap, packages/perseus/src/widgets/label-image/__stories__/label-image.stories.tsx, packages/perseus/src/widgets/label-image/__tests__/label-image.test.ts, packages/perseus/src/widgets/label-image/__tests__/label-image.testdata.ts, packages/perseus/src/widgets/matcher/__snapshots__/matcher.test.tsx.snap, packages/perseus/src/widgets/matrix/__snapshots__/matrix.test.ts.snap, packages/perseus/src/widgets/number-line/__snapshots__/number-line.test.ts.snap, packages/perseus/src/widgets/numeric-input/__snapshots__/numeric-input.test.ts.snap, packages/perseus/src/widgets/orderer/__snapshots__/orderer.test.ts.snap, packages/perseus/src/widgets/passage/__tests__/passage.test.tsx, packages/perseus/src/widgets/passage/__tests__/passage.testdata.ts, packages/perseus/src/widgets/passage-ref/__snapshots__/passage-ref.test.ts.snap, packages/perseus/src/widgets/python-program/__snapshots__/python-program.test.ts.snap, packages/perseus/src/widgets/radio/__stories__/base-radio.stories.tsx, packages/perseus/src/widgets/radio/__stories__/radio.stories.tsx, packages/perseus/src/widgets/radio/__tests__/base-radio.test.tsx, packages/perseus/src/widgets/radio/__tests__/base-radio.testdata.ts, packages/perseus/src/widgets/radio/__tests__/choice-icon.test.tsx, packages/perseus/src/widgets/radio/__tests__/choice.test.tsx, packages/perseus/src/widgets/radio/__tests__/focus-ring.test.tsx, packages/perseus/src/widgets/radio/__tests__/option-status.test.tsx, packages/perseus/src/widgets/radio/__tests__/radio.test.ts, packages/perseus/src/widgets/radio/__tests__/radio.testdata.ts, packages/perseus/src/widgets/video/__snapshots__/video.test.ts.snap, packages/perseus/src/widgets/passage/__tests__/__snapshots__/passage.test.tsx.snap, packages/perseus/src/widgets/radio/__tests__/__snapshots__/radio.test.ts.snap

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

@khan-actions-bot khan-actions-bot requested a review from a team September 4, 2024 18:45
Copy link
Contributor

github-actions bot commented Sep 4, 2024

npm Snapshot: Published

Good news!! We've packaged up the latest commit from this PR (e2392f3) and published it to npm. You
can install it using the tag PR1585.

Example:

yarn add @khanacademy/perseus@PR1585

If you are working in Khan Academy's webapp, you can run:

./dev/tools/bump_perseus_version.sh -t PR1585

Copy link
Member

@benchristel benchristel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for doing this!

@handeyeco handeyeco changed the title WIP: move widgets into widget-specific folders Move widgets into widget-specific folders Sep 4, 2024
Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.36%. Comparing base (21a908e) to head (e2392f3).
Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1585      +/-   ##
==========================================
+ Coverage   70.34%   70.36%   +0.01%     
==========================================
  Files         521      579      +58     
  Lines      107316   112070    +4754     
  Branches     5513    11275    +5762     
==========================================
+ Hits        75489    78853    +3364     
- Misses      31711    33217    +1506     
+ Partials      116        0     -116     

Impacted file tree graph

see 1100 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 615567b...e2392f3. Read the comment docs.

@handeyeco handeyeco merged commit a6ec402 into main Sep 4, 2024
19 of 30 checks passed
@handeyeco handeyeco deleted the move-widgets branch September 4, 2024 20:02
aemandine added a commit that referenced this pull request Sep 4, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @khanacademy/[email protected]

### Major Changes

- [#1577](#1577)
[`c875acd01`](c875acd)
Thanks [@handeyeco](https://github.com/handeyeco)! - Remove example
widgets and their editors

### Minor Changes

- [#1570](#1570)
[`c4432ffad`](c4432ff)
Thanks [@nicolecomputer](https://github.com/nicolecomputer)! - Remove
unlimited points via keyboard


- [#1582](#1582)
[`377b7ce68`](377b7ce)
Thanks [@aemandine](https://github.com/aemandine)! - Add save warnings
to PhET widget editor and un-hide widget from content editor widget
dropdown

### Patch Changes

- [#1578](#1578)
[`78bb8573e`](78bb857)
Thanks [@aemandine](https://github.com/aemandine)! - Remove
simpleValidate from PhET widget


- [#1585](#1585)
[`a6ec402c0`](a6ec402)
Thanks [@handeyeco](https://github.com/handeyeco)! - Reorganize files in
the widgets folder


- [#1589](#1589)
[`d56952564`](d569525)
Thanks [@aemandine](https://github.com/aemandine)! - Make PhET widget
smaller


- [#1587](#1587)
[`8015cdefb`](8015cde)
Thanks [@aemandine](https://github.com/aemandine)! - Tidying up PhET
widget


- [#1583](#1583)
[`615567bd2`](615567b)
Thanks [@handeyeco](https://github.com/handeyeco)! - Remove sort-comp
exceptions and reorder components

## @khanacademy/[email protected]

### Major Changes

- [#1577](#1577)
[`c875acd01`](c875acd)
Thanks [@handeyeco](https://github.com/handeyeco)! - Remove example
widgets and their editors

### Minor Changes

- [#1582](#1582)
[`377b7ce68`](377b7ce)
Thanks [@aemandine](https://github.com/aemandine)! - Add save warnings
to PhET widget editor and un-hide widget from content editor widget
dropdown

### Patch Changes

- [#1585](#1585)
[`a6ec402c0`](a6ec402)
Thanks [@handeyeco](https://github.com/handeyeco)! - Reorganize files in
the widgets folder


- [#1587](#1587)
[`8015cdefb`](8015cde)
Thanks [@aemandine](https://github.com/aemandine)! - Tidying up PhET
widget


- [#1583](#1583)
[`615567bd2`](615567b)
Thanks [@handeyeco](https://github.com/handeyeco)! - Remove sort-comp
exceptions and reorder components

- Updated dependencies
\[[`78bb8573e`](78bb857),
[`a6ec402c0`](a6ec402),
[`d56952564`](d569525),
[`c875acd01`](c875acd),
[`8015cdefb`](8015cde),
[`c4432ffad`](c4432ff),
[`615567bd2`](615567b),
[`377b7ce68`](377b7ce)]:
    -   @khanacademy/[email protected]

## @khanacademy/[email protected]

### Patch Changes

- [#1585](#1585)
[`a6ec402c0`](a6ec402)
Thanks [@handeyeco](https://github.com/handeyeco)! - Reorganize files in
the widgets folder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants