Salastil/DankMaterialShell
1
0
Fork 0
mirror of https://github.com/AvengeMedia/DankMaterialShell.git synced 2026-08-02 03:28:28 -04:00
Code Packages Releases Activity

fix(dash-tabs): update visibility logic for dash tab elements

Browse Source 
Fixes #2822

Port 1.5

(cherry picked from commit f590a2965a)
This commit is contained in:
purian23
2026-07-12 23:57:29 -04:00
committed by dms-ci[bot]
parent 6791c20f40
commit e29b9749df
3 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
quickshell/Common/SettingsData.qml
+2 -1
View File
@@ -617,7 +617,8 @@ Singleton {
function setDashTabEnabled(id, on) {
const current = getDashTabs();
if (!on && id !== "settings" && current.filter(t => t.enabled && t.id !== "settings").length <= 1)
const visibleContentIds = visibleDashTabIds().filter(tabId => tabId !== "settings");
if (!on && visibleContentIds.indexOf(id) >= 0 && visibleContentIds.length <= 1)
return;
dashTabs = current.map(t => t.id === id ? {
"id": t.id,
quickshell/Modules/DankDash/DankDashPopout.qml
+4 -2
View File
@@ -323,7 +323,8 @@ DankPopout {
id: tabBar
width: parent.width
height: 48
height: visible ? 48 : 0
visible: model.length > 0
currentIndex: root.currentTabIndex
spacing: Theme.spacingS
equalWidthTabs: true
@@ -355,7 +356,8 @@ DankPopout {
Item {
width: parent.width
height: Theme.spacingXS
height: visible ? Theme.spacingXS : 0
visible: tabBar.visible
}
Item {
quickshell/Modules/Settings/DankDashTab.qml
+2 -2
View File
@@ -48,7 +48,7 @@ Item {
// its delegates alive across commits (preserving focus for keyboard reorder)
readonly property var tabIds: SettingsData._dashTabIds
readonly property var tabState: SettingsData.getDashTabs()
readonly property int enabledContentCount: tabState.filter(t => t.enabled && t.id !== "settings").length
readonly property var visibleContentIds: SettingsData.visibleDashTabIds().filter(id => id !== "settings")
function presentationFor(id) {
return __presentation[id] ?? {
@@ -149,7 +149,7 @@ Item {
}
function canHide(id) {
return !isEnabled(id) || id === "settings" || enabledContentCount > 1;
return !isEnabled(id) || id === "settings" || visibleContentIds.indexOf(id) < 0 || visibleContentIds.length > 1;
}
// Keyboard nav is handled at the tab root (not per-row activeFocusOnTab)