Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Dropdown menu fixes #1361

Merged

Conversation

ascholerChemeketa
Copy link
Contributor

Fix two minor but visually annoying layout issues in top of page dropdowns.

Padding of items inside social menu ("Runestone in social media", "Help support us", and Donate button)
Display of search button next to text input inside search menu

@ascholerChemeketa
Copy link
Contributor Author

Note for Brad - adding one more fix to the dark/light slider... still in progress.

Fix accessibility by moving "Dark Mode" text inside label element
Cleanup layout - change size and indent to match other elements; add hover color to indicate interactivity
@ascholerChemeketa
Copy link
Contributor Author

Added the Dark Mode fix. Think this is done now.

@bnmnetp
Copy link
Member

bnmnetp commented Sep 6, 2022

Thanks, I will get it reviewed in the next day or two.

@bnmnetp bnmnetp merged commit 400346a into RunestoneInteractive:master Sep 6, 2022
@bnmnetp
Copy link
Member

bnmnetp commented Sep 6, 2022

Thanks! the dark mode toggle looks much nicer! I knew there had to be a way to do that.

I had not noticed that the search was broken either. I copied that change to the server code so its fixed there now too... Thanks

@ascholerChemeketa ascholerChemeketa deleted the dropdown-menu-fixes branch September 8, 2022 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants