Skip to content

Commit

Permalink
use the main theme for the sidebar and for the main page
Browse files Browse the repository at this point in the history
#540

@Ajotah98 : I'm open to a new pr to add a separate, documented, sidebar_theme, which falls back to the main theme when not specified
  • Loading branch information
lovasoa committed Aug 17, 2024
1 parent e156805 commit 969bba1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# CHANGELOG.md

## 0.27.0 (unreleased)
## 0.27.0 (2024-08-17)

- updated Apex Charts to v3.52.0
- see https://github.com/apexcharts/apexcharts.js/releases
Expand Down
2 changes: 1 addition & 1 deletion sqlpage/templates/shell.handlebars
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@
{{#if (or (or title (or icon image)) menu_item)}}
<header id="sqlpage_header">
{{#if sidebar}}
<aside class="navbar navbar-vertical navbar-expand-lg" {{#if sidebar-theme}}data-bs-theme="dark" {{/if}}>
<aside class="navbar navbar-vertical navbar-expand-lg" {{#if theme}}data-bs-theme="{{theme}}" {{/if}}>
<div class="container-fluid">
<button class="navbar-toggler collapsed" type="button" data-bs-target="#sidebar-menu" aria-controls="sidebar-menu" data-bs-toggle="collapse" aria-expanded="false" aria-label="Toggle navigation">
<span class="navbar-toggler-icon"></span>
Expand Down

0 comments on commit 969bba1

Please sign in to comment.