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

Addon-docs: DocsPage slots for fine-grained user control #7680

Merged
merged 1 commit into from
Aug 7, 2019

Conversation

shilman
Copy link
Member

@shilman shilman commented Aug 5, 2019

Issue: #7456

What I did

First cut at a "slots" feature for fine-grained control of what shows up on DocsPage, with sensible defaults.

What can be controlled:

  • title
  • subtitle
  • description
  • display of each story

Technically this is a breaking change because it modifies a public API. However, it's a mostly undocumented API that I don't expect anybody to use, and it's still beta. If you just used DocsPage out of the box, this change should be fully backwards-compatible.

How to test

See my update to official-storybook/config.js for an example usage

@vercel
Copy link

vercel bot commented Aug 5, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://monorepo-git-7456-docspage-slots.storybook.now.sh

Copy link
Member

@tmeasday tmeasday left a comment

Choose a reason for hiding this comment

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

LGTM. Not sure what the breaking change is though?

@shilman shilman requested a review from CodeByAlex August 6, 2019 02:57
Copy link
Member

@CodeByAlex CodeByAlex left a comment

Choose a reason for hiding this comment

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

Nice work! LGTM

@shilman shilman merged commit 4f9a846 into next Aug 7, 2019
@shilman shilman deleted the 7456-docspage-slots branch August 7, 2019 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants