Skip to content

Commit 5c6e5f7

Browse files
committed
fix: Fixed import mods button not working, b=(no-bug), c=mods, split-view
1 parent bd72aeb commit 5c6e5f7

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

src/zen/mods/actors/ZenModsMarketplaceChild.sys.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,8 @@ export class ZenModsMarketplaceChild extends JSWindowActorChild {
133133

134134
modId = button.getAttribute('zen-theme-id');
135135
} else {
136-
modId = event.themeId;
136+
// Backwards compatibility is... Interesting
137+
modId = event.themeId ?? event.modId ?? event.id;
137138
}
138139

139140
this.sendAsyncMessage('ZenModsMarketplace:InstallMod', { modId });

src/zen/split-view/ZenViewSplitter.mjs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1674,6 +1674,15 @@ class ZenViewSplitter extends ZenDOMOperatedFeature {
16741674
if (splitGroup && (!draggedTab.group || draggedTab.group !== splitGroup)) {
16751675
this._moveTabsToContainer([draggedTab], droppedOnTab);
16761676
gBrowser.moveTabToGroup(draggedTab, splitGroup);
1677+
if (hoverSide === 'left' || hoverSide === 'top') {
1678+
try {
1679+
splitGroup.tabs[0].before(draggedTab);
1680+
} catch (e) {
1681+
console.warn(
1682+
`Failed to move tab ${draggedTab.id} before ${splitGroup.tabs[0].id}: ${e}`
1683+
);
1684+
}
1685+
}
16771686
}
16781687

16791688
const droppedOnSplitNode = this.getSplitNodeFromTab(droppedOnTab);

0 commit comments

Comments
 (0)