Skip to content

Add 313-card-tab-focus #178

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

Merged
merged 14 commits into from
May 19, 2023
Merged

Add 313-card-tab-focus #178

merged 14 commits into from
May 19, 2023

Conversation

gadenbuie
Copy link
Member

@gadenbuie gadenbuie commented May 17, 2023

Overview

The primary goal of this PR is to test rstudio/bslib#557. The app contains three full-screen expandable cards. The first has no focusable elements, the second has many input plus two child cards, and the third includes a plotly plot.

The test cycle exercise the behavior around entering and exiting full screen mode as well as the management of tab focus while in the full screen card.

Demo

Here's a short video walking through the app

Screenshare.-.2023-05-17.2_33_20.PM.mp4

Todo

  • Need to test that pressing escape while focused on a select element doesn't exit full screen mode

@gadenbuie gadenbuie requested review from schloerke and cpsievert May 17, 2023 18:38
@gadenbuie gadenbuie self-assigned this May 17, 2023
@gadenbuie gadenbuie marked this pull request as ready for review May 18, 2023 17:12
Copy link
Contributor

@cpsievert cpsievert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

#Conflicts:
#	inst/apps/sysinfo-mac-4.1.txt
#	inst/apps/sysinfo-mac-4.3.txt
#	inst/apps/sysinfo-win-3.6.txt
#	inst/apps/sysinfo-win-4.0.txt
#	inst/apps/sysinfo-win-4.1.txt
#	inst/apps/sysinfo-win-4.2.txt
#	inst/apps/sysinfo-win-4.3.txt
@gadenbuie gadenbuie merged commit 1ee38c1 into main May 19, 2023
@gadenbuie gadenbuie deleted the 313-card-tab-focus branch May 19, 2023 15:46
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.

3 participants