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

725 fix get button #865

Merged
merged 3 commits into from
Oct 3, 2023
Merged

725 fix get button #865

merged 3 commits into from
Oct 3, 2023

Conversation

manuq
Copy link
Collaborator

@manuq manuq commented Oct 3, 2023

Helps #725

manuq added 3 commits October 3, 2023 11:45
We no longer have different colors per content kind (video, topic, etc)
so the theme colors were all the same gray. Make this clear with a "gray"
theme color and also add the endless-orange to this color. So b-buttons
can now have variant=gray-primary or variant=endless-orange-outline (for
example).

Helps: #725
Use a bootstrap flexbox utility class to center the button vertically in
the navbar without stretching it.

https://getbootstrap.com/docs/4.0/utilities/flex/#align-self
@manuq manuq requested a review from jprvita October 3, 2023 16:16
@jprvita jprvita merged commit 637be03 into master Oct 3, 2023
@jprvita jprvita deleted the 725-fix-get-button branch October 3, 2023 17:09
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