Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[ENHANCEMENT] Improve button display in menus #509

Closed
nicolepaschen opened this issue Sep 28, 2021 · 4 comments · Fixed by #520
Closed

[ENHANCEMENT] Improve button display in menus #509

nicolepaschen opened this issue Sep 28, 2021 · 4 comments · Fixed by #520
Labels
Milestone

Comments

@nicolepaschen
Copy link
Member

I have a menu item I've added the custom css class button-primary to so I have a button in my menu:
image

There is too much padding on the sides and top at this time:
image

@nicolepaschen
Copy link
Member Author

Please update the hover effects to be the same as the standard primary and secondary buttons.

menu-button-hover

@nicolepaschen
Copy link
Member Author

I have a button on the Contact Us menu item. When I am on the Contact Us page, the button no longer appears. It should stay there so the header looks the same across the site.

menu-button-active

@nicolepaschen
Copy link
Member Author

I started with the Wedge Inspiration. I changed the menu to have a single color hover effect. There are two buttons in the menu coded with button and button-primary/secondary:

image

The single color hover effect is applied to the buttons as well. The buttons should always use the standard hover effect please.

button-hover-effect-menu

@nicolepaschen
Copy link
Member Author

I started with Starter Content and changed the color palette. I added primary and secondary buttons to the header. I changed the hover style to a single color and changed the link color of the menu items. The buttons inherit the link color of menu items.

buttons

@jamesros161 jamesros161 added the Feature Request Feature Request label Nov 16, 2021
@jamesros161 jamesros161 added this to the Stable milestone Apr 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants