search

ThemeAwesome / WP-Forge

WP-Forge - A Foundation for Sites WordPress Theme
https://themeawesome.com/wp-forge-wordpress-foundation-for-sites-theme/
85 stars 28 forks source link

Top-Bar Link Active Color #50

Closed ghost closed 7 years ago

ghost commented 8 years ago

Hi,

there's currently a customizer color option "Top-Bar Active Color" but no "Top-Bar Link Active Color" that would set the color of .menu .active > a.

For example a black top bar background and white link color together with white active link background hides the text of the active link.

Could the color option "Top-Bar Link Active Color" be added? (For my purposes "Top-Bar Link Hover Color" could be reused to set the color of .menu .active > a but I imagine that these two should be separated in some cases.)

tsquez commented 8 years ago

Hey there,

Def this can be done amigo and I have been wanting to do this for a while now but I keep forgetting.

I'll make this part of the next update. Thanks again for mentioning it. Its greatly appreciated.

tsquez commented 7 years ago

Almost done with the 6.3 update, I hope to have this included in the update. Stay tuned. Thanks again for the input.

tsquez commented 7 years ago

Actually, removing Foundation active class from WP-Forge. I have opted to go with the built in current-menu-item that is built into WordPress. This is easier to style and looks a lot cleaner. See this post https://developer.wordpress.org/reference/functions/wp_nav_menu/

I think it looks great and I hope you do to :wink:

ghost commented 7 years ago

Sounds good. Thank you!