From 22c29ebf0d9734e0bdcbd47a737e0d684dd41afc Mon Sep 17 00:00:00 2001 From: Garrick Aden-Buie Date: Tue, 2 Jan 2024 13:28:53 -0500 Subject: [PATCH] fix(311-sidebar-toggle): Set `tabindex=0` on ui output parent element `uiOutput()` elements now get `display:contents`, which means they can't receive tab focus. The tabindex is used in 311 to test correct tab order, not for any real functionality in the app. --- inst/apps/311-bslib-sidebar-toggle-methods/app.R | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/inst/apps/311-bslib-sidebar-toggle-methods/app.R b/inst/apps/311-bslib-sidebar-toggle-methods/app.R index f2f0003886..dc273aee82 100644 --- a/inst/apps/311-bslib-sidebar-toggle-methods/app.R +++ b/inst/apps/311-bslib-sidebar-toggle-methods/app.R @@ -64,7 +64,7 @@ ui <- page_fixed( border = FALSE, border_radius = FALSE, h2("Sidebar Layout"), - uiOutput("ui_content", tabindex = 0), + div(uiOutput("ui_content"), tabindex = 0), ) ) ),