Skip to content

Commit 29f7af7

Browse files
committed
fix: Make use of built-in folder query methods
1 parent c681e5a commit 29f7af7

2 files changed

Lines changed: 19 additions & 2 deletions

File tree

src/api/api.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,8 @@ this.quickmove = class extends ExtensionAPI {
2929
return {
3030
quickmove: {
3131
// bug 1840039 - messenger.folders.query API
32-
// Keeping this around as this verison allows us to switch between MRMTime and MRUTime
32+
// bug 1945514 - allow differing between MRU/MRMTime
33+
// TB136 COMPAT
3334
async query({ recent, limit, canFileMessages }) {
3435
function* allFolders(root) {
3536
if (

src/popup/popup.js

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ function switchList(action) {
3232
}
3333

3434
async function load() {
35+
let browserInfo = await browser.runtime.getBrowserInfo();
36+
let majorVersion = parseInt(browserInfo.version.split(".")[0], 10);
37+
3538
let fontSize = await messenger.quickmove.getUIFontSize();
3639
window.document.documentElement.style.setProperty("font-size", `${fontSize}px`);
3740

@@ -129,7 +132,20 @@ async function load() {
129132
let defaultFolders;
130133

131134
if (defaultFolderSetting == "recent") {
132-
let folderList = await browser.quickmove.query({ recent: recentStrategy, limit: maxRecentFolders, canFileMessages: true });
135+
let folderList;
136+
if (majorVersion < 137) {
137+
// TB136 COMPAT
138+
folderList = await browser.quickmove.query({ recent: recentStrategy, limit: maxRecentFolders, canFileMessages: true });
139+
} else {
140+
let lastProperty = recentStrategy == "modified" ? "lastUsedAsDestination" : "lastUsed";
141+
folderList = await browser.folders.query({
142+
limit: browser.folder.DEFAULT_MOST_RECENT_LIMIT,
143+
[lastProperty]: { recent: true },
144+
sort: lastProperty,
145+
canAddMessages: true
146+
});
147+
}
148+
133149
defaultFolders = rootNode.fromList(folderList).folderNodes;
134150
} else if (defaultFolderSetting == "specific") {
135151
defaultFolders = await getValidatedFolders(rootNode, "defaultFolders");

0 commit comments

Comments
 (0)