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

Mark dropdown with active class correctly, #372

Merged
merged 5 commits into from
Oct 1, 2021
Merged

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert requested a review from wch October 1, 2021 19:58
@cpsievert cpsievert merged commit ea30ec5 into master Oct 1, 2021
@cpsievert cpsievert deleted the dropdown-active branch October 1, 2021 20:46
cpsievert added a commit to rstudio/shiny that referenced this pull request Oct 1, 2021
cpsievert added a commit to rstudio/shiny that referenced this pull request Jan 12, 2023
* Add snapshot test for #3519 which was fixed via rstudio/bslib#372

* sync package version (GitHub Actions)

* yarn build (GitHub Actions)

* `yarn build` (GitHub Actions)

* Sync package version (GitHub Actions)

Co-authored-by: cpsievert <cpsievert@users.noreply.github.com>
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.

navbarMenu() aren't being selected properly on pageload
1 participant