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

Navlist vendor prefix #214

Merged
merged 4 commits into from
Jan 8, 2021
Merged

Navlist vendor prefix #214

merged 4 commits into from
Jan 8, 2021

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Jan 7, 2021

For a shinytest specific issue found in rstudio/shinycoreci-apps#77

(Also cleans-up patch file from #213)

cpsievert added a commit to rstudio/shinycoreci-apps that referenced this pull request Jan 7, 2021
cpsievert added a commit to rstudio/shinycoreci-apps that referenced this pull request Jan 7, 2021
@cpsievert cpsievert requested a review from wch January 8, 2021 16:56
@cpsievert cpsievert merged commit 966c26e into master Jan 8, 2021
@cpsievert cpsievert deleted the navlist-vendor-prefix branch January 8, 2021 17:08
cpsievert added a commit to rstudio/shinycoreci-apps that referenced this pull request Jan 8, 2021
* init dynamic-tabs-compat testing app

* Focus app logic and tests on insert+remove+show+hide of tabs in navbarPage()+tabsetPanel()+navlistPanel()

* Run tests with BS3 and BS4

* Approve corrected BS4 screenshots (with rstudio/bslib#214)

* Remove show/hide from navbar controls
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