From ba767e71a88fdad0a226ba35b50b58487f011073 Mon Sep 17 00:00:00 2001
From: Matt Dray <18232097+matt-dray@users.noreply.github.com>
Date: Thu, 5 Feb 2026 17:35:42 +0000
Subject: [PATCH] Rearrange cards in visulisations tab

---
 R/app_ui.R         | 32 ++++++++++----------------------
 R/mod_plot_rates.R |  2 --
 2 files changed, 10 insertions(+), 24 deletions(-)

diff --git a/R/app_ui.R b/R/app_ui.R
index 388f0db..74c81f4 100644
--- a/R/app_ui.R
+++ b/R/app_ui.R
@@ -11,9 +11,7 @@ app_ui <- function(request) {
     bslib::nav_panel(
       id = "nav_panel_viz",
       title = "Visualisations",
-
       bslib::card(
-        fill = FALSE,
         bslib::card_header(
           class = "bg-warning",
           bsicons::bs_icon("exclamation-triangle"),
@@ -22,27 +20,17 @@ app_ui <- function(request) {
         "This app is in development and its output has not been verified.",
         "The information presented here should not be relied on as fact."
       ),
-
+      mod_show_strategy_text_ui("mod_show_strategy_text"),
+      mod_plot_rates_ui("mod_plot_rates"),
       bslib::layout_columns(
-        col_widths = c(3, 9),
-        fill = FALSE,
-        fillable = FALSE,
-        mod_show_strategy_text_ui("mod_show_strategy_text"),
-        bslib::layout_column_wrap(
-          width = 1,
-          gap = "0.8rem",
-          mod_plot_rates_ui("mod_plot_rates"),
-          bslib::layout_column_wrap(
-            width = 1 / 2,
-            mod_table_procedures_ui("mod_table_procedures"),
-            mod_table_diagnoses_ui("mod_table_diagnoses")
-          ),
-          bslib::layout_column_wrap(
-            width = 1 / 2,
-            mod_plot_age_sex_pyramid_ui("mod_plot_age_sex_pyramid"),
-            mod_plot_nee_ui("mod_plot_nee")
-          ),
-        )
+        col_widths = c(6, 6),
+        mod_table_diagnoses_ui("mod_table_diagnoses"),
+        mod_table_procedures_ui("mod_table_procedures")
+      ),
+      bslib::layout_columns(
+        col_widths = c(6, 6),
+        mod_plot_age_sex_pyramid_ui("mod_plot_age_sex_pyramid"),
+        mod_plot_nee_ui("mod_plot_nee")
       )
     ),
 
diff --git a/R/mod_plot_rates.R b/R/mod_plot_rates.R
index 2336963..c3d9f7b 100644
--- a/R/mod_plot_rates.R
+++ b/R/mod_plot_rates.R
@@ -6,8 +6,6 @@ mod_plot_rates_ui <- function(id) {
   # Rates plots share a y-axis, so don't wrap
   bslib::layout_columns(
     col_widths = c(5, 5, 2),
-    fill = FALSE,
-    fillable = FALSE,
     mod_plot_rates_trend_ui(ns("mod_plot_rates_trend")),
     mod_plot_rates_funnel_ui(ns("mod_plot_rates_funnel")),
     mod_plot_rates_box_ui(ns("mod_plot_rates_box"))