Merge pull request #38914 from KerollesFathy/fix-getting-started
fix(sidebar): show "Getting Started" only when sidebar expanded
This commit is contained in:
commit
242e7b55ca
2 changed files with 3 additions and 1 deletions
|
|
@ -42,7 +42,7 @@
|
|||
</div>
|
||||
<div class="promotional-banners"></div>
|
||||
<p>
|
||||
<a class="onboarding-sidebar">
|
||||
<a class="onboarding-sidebar px-2">
|
||||
{%= frappe.utils.icon("user-check" , "sm", "", "", "text-ink-gray-7 current-color", true)%}
|
||||
<span> {%= __("Getting Started") %} </span>
|
||||
</a>
|
||||
|
|
|
|||
|
|
@ -582,6 +582,7 @@ frappe.ui.Sidebar = class Sidebar {
|
|||
$('[data-toggle="tooltip"]').tooltip("dispose");
|
||||
this.wrapper.find(".avatar-name-email").show();
|
||||
this.wrapper.find(".about-sidebar-link").show();
|
||||
this.wrapper.find(".onboarding-sidebar span").show();
|
||||
} else {
|
||||
this.wrapper.removeClass("expanded");
|
||||
direction = "left";
|
||||
|
|
@ -592,6 +593,7 @@ frappe.ui.Sidebar = class Sidebar {
|
|||
});
|
||||
this.wrapper.find(".avatar-name-email").hide();
|
||||
this.wrapper.find(".about-sidebar-link").hide();
|
||||
this.wrapper.find(".onboarding-sidebar span").hide();
|
||||
}
|
||||
|
||||
localStorage.setItem("sidebar-expanded", this.sidebar_expanded);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue