refactor: improve platform-specific button handling and streamline profile opening logic

This commit is contained in:
mr. M
2024-12-10 17:00:12 +01:00
parent 900313bbef
commit 15ff4f12ac
3 changed files with 4 additions and 8 deletions

View File

@@ -202,7 +202,7 @@ var gZenVerticalTabsManager = {
get actualWindowButtons() { get actualWindowButtons() {
// we have multiple ".titlebar-buttonbox-container" in the DOM, because of the titlebar // we have multiple ".titlebar-buttonbox-container" in the DOM, because of the titlebar
if (!this.__actualWindowButtons) { if (!this.__actualWindowButtons) {
this.__actualWindowButtons = AppConstants.platform === 'macosx' ? this.__actualWindowButtons = (AppConstants.platform === 'macosx' || this.isWindowsStyledButtons) ?
document.querySelector('.titlebar-buttonbox-container') : // TODO: test if it works 100% of the time document.querySelector('.titlebar-buttonbox-container') : // TODO: test if it works 100% of the time
document.querySelector('#nav-bar .titlebar-buttonbox-container'); document.querySelector('#nav-bar .titlebar-buttonbox-container');
} }

View File

@@ -49,11 +49,7 @@ var ZenProfileDialogUI = {
}, },
_openProfile(profile) { _openProfile(profile) {
SelectableProfileService.getProfile( Services.startup.createInstanceWithProfile(profile);
aEvent.target.getAttribute("profileid")
).then(profile => {
SelectableProfileService.launchInstance(profile);
});
}, },
_getProfilesSize(profiles) { _getProfilesSize(profiles) {

View File

@@ -430,11 +430,11 @@ var ZenWorkspaces = new (class extends ZenMultiWindowFeature {
let activeWorkspace = await this.getActiveWorkspace(); let activeWorkspace = await this.getActiveWorkspace();
if (!activeWorkspace) { if (!activeWorkspace) {
activeWorkspace = workspaces.workspaces.find((workspace) => workspace.default); activeWorkspace = workspaces.workspaces.find((workspace) => workspace.default);
this.activeWorkspace = activeWorkspace.uuid; this.activeWorkspace = activeWorkspace?.uuid;
} }
if (!activeWorkspace) { if (!activeWorkspace) {
activeWorkspace = workspaces.workspaces[0]; activeWorkspace = workspaces.workspaces[0];
this.activeWorkspace = activeWorkspace.uuid; this.activeWorkspace = activeWorkspace?.uuid;
} }
await this.changeWorkspace(activeWorkspace, true); await this.changeWorkspace(activeWorkspace, true);
} }