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

docs(banner): update blue colors to cyan & fix button paddings #957

Merged
merged 2 commits into from
Sep 13, 2023

Conversation

zoltanszogyenyi
Copy link
Member

@zoltanszogyenyi zoltanszogyenyi commented Sep 13, 2023

I've checked the banner component and I'd like to do the following changes:

  • update the blue colors to cyan
  • the X icon should have even paddings (ie. px-2 py-2 or p-2 instead of px-4 py-2)
  • responsiveness issues for some of the banners (marketing cta banner, bottom banner)

The button is too large, we should make it smaller:

Screenshot 2023-09-13 at 11 04 29

The elements should span the whole width of the screen:

Screenshot 2023-09-13 at 11 04 21

--

For reference we can check out the banners from the main Flowbite Library.

Screenshot 2023-09-13 at 11 06 15 Screenshot 2023-09-13 at 11 06 10

@vercel
Copy link

vercel bot commented Sep 13, 2023

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

Name Status Preview Comments Updated (UTC)
flowbite-react ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 13, 2023 8:10am

@rluders rluders merged commit 4578c1c into main Sep 13, 2023
@rluders rluders deleted the banner-fixes branch September 13, 2023 08:25
@zoltanszogyenyi zoltanszogyenyi restored the banner-fixes branch September 14, 2023 06:44
@zoltanszogyenyi zoltanszogyenyi deleted the banner-fixes branch September 14, 2023 06:45
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.

2 participants