TO recreate: 1. Have some reports and report groups. 2. GO to Reports > Used Saved 3. The 'All' tab properly shows reports. 4. All other tabs have no reports listed. It doesn't appear like the panels are being built. Looks to be caused by Bug 35402.
I'm not sure what got lost. It seems like there was only a single panel being built 'reports_panel' ( [% WRAPPER tab_panels %] ). However, when building the nav-links all the groups are looped through ( [% FOREACH group IN groups_with_subgroups %] ). ( [% WRAPPER tabs_nav %] ) We cannot build many tabs and only a single panel. It creates bad HTML.
Created attachment 170838 [details] [review] Bug 37740: Fix saved SQL reports tab switching The tabs on the saved SQL report page are not really full tabs, they're just styled that way. There is only one tab "panel," and clicking each tab triggers a DataTable filter so that the tab only shows the relevant data. The problem with this setup is that when Bootstrap tries to "switch" tabs, it wants to hide the initial tab and then show the new one. There is no "new" one here. The attached patch solves this problem by adding a CSS rule to guided_reports_start.tt with higher specificity than our global CSS making the relevant tab pane visible at all times. To test, apply the patch and go to Reports -> Use saved. - If necessary, create multiple SQL reports and assign them to different report groups (Accounts, Acquisitions, Catalog, etc.). - Test that tab switching works: - The tab should be style as "active" after you click it. - The table of reports should be filtered to include only reports in the relevant report group.
Created attachment 170871 [details] [review] Bug 37740: Fix saved SQL reports tab switching The tabs on the saved SQL report page are not really full tabs, they're just styled that way. There is only one tab "panel," and clicking each tab triggers a DataTable filter so that the tab only shows the relevant data. The problem with this setup is that when Bootstrap tries to "switch" tabs, it wants to hide the initial tab and then show the new one. There is no "new" one here. The attached patch solves this problem by adding a CSS rule to guided_reports_start.tt with higher specificity than our global CSS making the relevant tab pane visible at all times. To test, apply the patch and go to Reports -> Use saved. - If necessary, create multiple SQL reports and assign them to different report groups (Accounts, Acquisitions, Catalog, etc.). - Test that tab switching works: - The tab should be style as "active" after you click it. - The table of reports should be filtered to include only reports in the relevant report group. Signed-off-by: Chloe Zermatten <chloe.zermatten@ptfs-europe.com>
Created attachment 170879 [details] [review] Bug 37740: Fix saved SQL reports tab switching The tabs on the saved SQL report page are not really full tabs, they're just styled that way. There is only one tab "panel," and clicking each tab triggers a DataTable filter so that the tab only shows the relevant data. The problem with this setup is that when Bootstrap tries to "switch" tabs, it wants to hide the initial tab and then show the new one. There is no "new" one here. The attached patch solves this problem by adding a CSS rule to guided_reports_start.tt with higher specificity than our global CSS making the relevant tab pane visible at all times. To test, apply the patch and go to Reports -> Use saved. - If necessary, create multiple SQL reports and assign them to different report groups (Accounts, Acquisitions, Catalog, etc.). - Test that tab switching works: - The tab should be style as "active" after you click it. - The table of reports should be filtered to include only reports in the relevant report group. Signed-off-by: Chloe Zermatten <chloe.zermatten@ptfs-europe.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Pushed for 24.11! Well done everyone, thank you!