Docker/Maybe
Docker/Maybe
1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-08-10 07:55:21 +02:00
Code Issues Releases Activity Actions

Fix sidebar mobile conflict

Browse source
This commit is contained in:
Zach Gollwitzer 2025-04-09 12:37:18 -04:00
parent 3c45d0a74a
commit 74e874a384
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
app/javascript/controllers/sidebar_controller.js
View file

@ -7,7 +7,7 @@ export default class extends Controller {
config: Object,
};
static targets = ["leftPanel", "rightPanel", "content"];
static targets = ["leftPanel", "leftPanelMobile", "rightPanel", "content"];
initialize() {
this.leftPanelOpen = this.configValue.left_panel.is_open;
@ -22,10 +22,10 @@ export default class extends Controller {
toggleLeftPanelMobile() {
if (this.leftPanelOpen) {
this.leftPanelTarget.classList.add("hidden");
this.leftPanelMobileTarget.classList.remove("hidden");
this.leftPanelOpen = false;
} else {
this.leftPanelTarget.classList.remove("hidden");
this.leftPanelMobileTarget.classList.add("hidden");
this.leftPanelOpen = true;
}
}

2
app/views/layouts/application.html.erb
View file

@ -13,7 +13,7 @@
<%# Mobile only account sidebar groups %>
<%= tag.div class: class_names("hidden bg-gray-50 z-20 absolute inset-0 h-dvh w-full p-4 overflow-y-auto transition-all duration-300"),
data: { sidebar_target: "leftPanel" } do %>
data: { sidebar_target: "leftPanelMobile" } do %>
<div id="account-sidebar-tabs">
<div class="mb-4">
<button data-action="sidebar#toggleLeftPanelMobile">