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

Blocks: Add of prop to Subtitle #22552

Merged
merged 18 commits into from
Apr 22, 2024
Merged

Conversation

joaonunomota
Copy link
Contributor

@joaonunomota joaonunomota commented May 15, 2023

Closes part of #22490

What I did

  • Add support for of prop via useOf with "meta" as only valid type
  • Default to extracting content from parameters.docs.subtitle
  • Deprecate extracting content from parameters.componentSubtitle
  • Add stories for several possible applications of Subtitle
  • Update docs and migration files
  • Fix typo in EmptyExample

How to test

  1. Run yarn storybook:blocks
  2. Open Storybook in your browser
  3. Access Subtitle stories

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests)
  • Make sure to add/update documentation regarding your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Maintainers

  • If this PR should be tested against many or all sandboxes,
    make sure to add the ci:merged or ci:daily GH label to it.
  • Make sure this PR contains one of the labels below.

["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

@@ -24,10 +24,16 @@ import { Subtitle } from '@storybook/blocks';

`Subtitle` is configured with the following props:

### `of`
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for updating the API reference! ❤️

Could you please keep these props in alphabetical order?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorted :)

@joaonunomota joaonunomota requested a review from kylegach August 7, 2023 22:58
@joaonunomota joaonunomota changed the title Upgrade Subtitle block to the new v7 blocks API Doc Blocks: Add support for of prop to Subtitle Oct 27, 2023
@joaonunomota
Copy link
Contributor Author

@JReinhold Hi! Is there anything that I can update / fix for a pull request review? It would be nice if this PR and #23728 could go out with #23849.

@JReinhold
Copy link
Contributor

Sorry @joaonunomota I've missed this. I'll review this early next week

@ndelangen ndelangen changed the title Doc Blocks: Add support for of prop to Subtitle Blocks: Add support for of prop to Subtitle Nov 28, 2023
@ndelangen
Copy link
Member

@JReinhold please review?

Copy link

nx-cloud bot commented Apr 22, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 1c204ea. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Copy link
Contributor

@JReinhold JReinhold left a comment

Choose a reason for hiding this comment

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

Thank you for the fantastic work, I'm sorry it took so long for me to properly review it. <3

@JReinhold JReinhold merged commit 2bff7a1 into storybookjs:next Apr 22, 2024
49 of 51 checks passed
@github-actions github-actions bot mentioned this pull request May 3, 2024
44 tasks
@shilman shilman changed the title Blocks: Add support for of prop to Subtitle Blocks: Add of prop to Subtitle May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants