Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
023e5c4
Remove "experimental" designation for `ProgressBar`
fullofcaffeine Apr 19, 2024
b1fe7cd
Add optional width prop to override/set the progress bar track container
fullofcaffeine Apr 25, 2024
fe9129d
Revert "Add optional width prop to override/set the progress bar trac…
fullofcaffeine May 22, 2024
64c72e2
Keep an unconstrained width by default, while allowing consumers to o…
fullofcaffeine May 22, 2024
c88ece0
Revert "Keep an unconstrained width by default, while allowing consum…
fullofcaffeine May 24, 2024
ad0fff9
Make the component public
fullofcaffeine May 24, 2024
dc299ed
Update consumers to import the public component
fullofcaffeine May 24, 2024
4c0b8fd
Expose docs for the now public ProgressBar component
fullofcaffeine May 24, 2024
0ff6637
Update CHANGELOG
fullofcaffeine May 24, 2024
ad2fd28
Merge branch 'trunk' into remove/wp-components-experimental-designati…
fullofcaffeine May 24, 2024
adf39a1
Run docs:build after docs/manifest.js change
fullofcaffeine May 24, 2024
ceb2840
Merge branch 'trunk' into remove/wp-components-experimental-designati…
fullofcaffeine May 24, 2024
9b3db94
Remove remaining private usage of ProgressBar
fullofcaffeine May 24, 2024
a219fa8
Small formatting fix in the changelog
fullofcaffeine May 24, 2024
1d22f24
Add basic docs to the README
fullofcaffeine May 24, 2024
b096ca4
Add jsdoc
fullofcaffeine May 24, 2024
3af0985
Formatting fix in README jsx block
fullofcaffeine May 27, 2024
846ccc0
Another formatting fix in README jsx block
fullofcaffeine May 27, 2024
39b92a5
YAFF (yet another formatting fix) in the README jsx
fullofcaffeine May 27, 2024
3241379
Improve wording in descripton of the `value` prop in the README
fullofcaffeine May 27, 2024
ff982fb
YAFF: Missing semicolon in README jsx
fullofcaffeine May 27, 2024
b0eca3f
YAFF: missing semicolon in jsx block in README
fullofcaffeine May 27, 2024
59d46cc
YAFF: missing semicolon in jsx block in README
fullofcaffeine May 27, 2024
0832e40
YAFF: use tab instead of spaces in jsx code block in README
fullofcaffeine May 27, 2024
5ca947a
Move export out of the HOC export section
fullofcaffeine May 27, 2024
47183b7
Fix CHANGELOG: Move entry to the existing enhacements section
fullofcaffeine May 27, 2024
eed1dba
Merge branch 'trunk' into remove/wp-components-experimental-designati…
fullofcaffeine May 27, 2024
d132170
Spacing fix
fullofcaffeine May 28, 2024
43e1a57
Sort import alphabetically
fullofcaffeine May 28, 2024
7a00da3
Allow `ProgressBar` to have unconstrained width, which is disabled by…
fullofcaffeine May 24, 2024
a9c174b
Revert "Allow `ProgressBar` to have unconstrained width, which is dis…
fullofcaffeine May 28, 2024
66e8223
Update docs and add story to illustrate a custom width
fullofcaffeine May 28, 2024
3a95761
Update CHANGELOG
fullofcaffeine May 28, 2024
a076910
Merge branch 'trunk' into improve/progress-bar-width-control
fullofcaffeine May 29, 2024
2454c7a
Simplify description in the story
fullofcaffeine May 29, 2024
981ce6f
Improve jsdoc in story, do not encourage more customization than needed
fullofcaffeine May 29, 2024
67d97df
Inherit the second story from the main template
fullofcaffeine May 29, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Inherit the second story from the main template
  • Loading branch information
fullofcaffeine committed May 29, 2024
commit 67d97df116c87ba655d20a144596e02b4b826c88
14 changes: 9 additions & 5 deletions packages/components/src/progress-bar/stories/index.story.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,15 @@ const withCustomWidthCustomCSS = `
* }
* ```
*/
export const WithCustomWidth: StoryFn< typeof ProgressBar > = () => {
return (
export const WithCustomWidth = Template.bind( {} );
WithCustomWidth.args = {
className: 'custom-progress-bar',
};
WithCustomWidth.decorators = [
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Another thing that's not specific to this PR, but I wonder why we're still using a legacy CSF version, even for new stuff. Haven't checked our current version, but CSF 3 has been supported for the last couple of versions and it makes Storybook much easier to work with. @WordPress/gutenberg-components

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We probably should! Nobody hadn't really prioritized it enough.

( Story ) => (
<>
<style>{ withCustomWidthCustomCSS }</style>
<ProgressBar className="custom-progress-bar" />
<Story />
</>
);
};
),
];