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

Added option to see a brand note in the footer #79

Merged
merged 6 commits into from
Nov 3, 2022

Conversation

achampagnedev
Copy link
Contributor

@achampagnedev achampagnedev commented Nov 1, 2022

Project organization

Description

The goal of this PR is to add the option to see a brand note "Powered By" in the Footer
image

added property showBrandNote to footer to enable this functionality.

How to test

See Storybook in the Footer section story With Brand Note

https://storybook-components-clsttr3th-liteflow.vercel.app/?path=/story/templates-footer-footer--with-brand-note

Checklist

  • Update related changelogs

@achampagnedev achampagnedev self-assigned this Nov 1, 2022
@vercel
Copy link

vercel bot commented Nov 1, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 2, 2022 at 3:02AM (UTC)
storybook-components ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 2, 2022 at 3:02AM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
storybook-templates ⬜️ Ignored (Inspect) Nov 2, 2022 at 3:02AM (UTC)

@vercel vercel bot temporarily deployed to Preview – storybook-templates November 1, 2022 13:25 Inactive
@vercel vercel bot temporarily deployed to Preview – docs November 1, 2022 13:26 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-components November 1, 2022 13:28 Inactive
@vercel vercel bot temporarily deployed to Preview – docs November 1, 2022 13:29 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-templates November 1, 2022 13:30 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-components November 1, 2022 13:34 Inactive
@vercel vercel bot temporarily deployed to Preview – docs November 1, 2022 13:35 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-templates November 1, 2022 13:40 Inactive
Copy link
Member

@antho1404 antho1404 left a comment

Choose a reason for hiding this comment

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

If design is good for @EmmanuelDrouin it's good for me.
Naming might be improved showBrandNote, I would have put something more like showPoweredBy so there is no mental conversion but not a big issue

@achampagnedev
Copy link
Contributor Author

Just switched naming to showPoweredBy

@vercel vercel bot temporarily deployed to Preview – storybook-templates November 2, 2022 00:38 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-components November 2, 2022 00:40 Inactive
@vercel vercel bot temporarily deployed to Preview – docs November 2, 2022 00:43 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-components November 2, 2022 02:56 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-templates November 2, 2022 02:59 Inactive
@vercel vercel bot temporarily deployed to Preview – docs November 2, 2022 03:00 Inactive
@vercel vercel bot temporarily deployed to Preview – storybook-components November 2, 2022 03:02 Inactive
@EmmanuelDrouin
Copy link
Member

If design is good for @EmmanuelDrouin it's good for me.

Yep design is good for me

@antho1404 antho1404 merged commit 9d3c6f7 into main Nov 3, 2022
@antho1404 antho1404 deleted the feature/add-footer-brand-note branch November 3, 2022 05:17
@EmmanuelDrouin
Copy link
Member

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