-
Notifications
You must be signed in to change notification settings - Fork 355
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
feat(react-core/bullseye): added Bullseye Layout #435
feat(react-core/bullseye): added Bullseye Layout #435
Conversation
Deploy preview for patternfly-react ready! Built with commit 14d0039 |
package.json
Outdated
"storybook:run": "start-storybook -c storybook -p 6006", | ||
"storybook:build": "build-storybook -c storybook -o .out", | ||
"storybook:run": "yarn build && start-storybook -c storybook -p 6006", | ||
"storybook:build": "yarn build && build-storybook -c storybook -o .out", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this intended as part of this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can separate it out if needed, but yes. Seems people are still having issues with running storybook and not building first.
14d0039
to
c5d8e0a
Compare
Pull Request Test Coverage Report for Build 1656
💛 - Coveralls |
can you add some snapshots? I was looking for the rendered markup... |
affects: @patternfly/react-core, @patternfly/react-docs Added Bullseye layout from here: https://pf-next.com/layouts/Bullseye/examples/ ISSUES CLOSED: patternfly#434
c5d8e0a
to
0135f46
Compare
affects: @patternfly/react-core, @patternfly/react-docs
Added Bullseye layout from here: https://pf-next.com/layouts/Bullseye/examples/
ISSUES CLOSED: #434