Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recent Folder does not update until TB closed and reopened #76

Closed
swolh opened this issue Nov 23, 2020 · 5 comments
Closed

Recent Folder does not update until TB closed and reopened #76

swolh opened this issue Nov 23, 2020 · 5 comments

Comments

@swolh
Copy link

swolh commented Nov 23, 2020

The drop down list of recently opened folders that shows up when you click on the "Recent Folders" tab does not update after you've opened or moved a message to a folder. It seems to only update by closing TB and reopening TB.

@RealRaven2000
Copy link
Owner

The drop down list of recently opened folders that shows up when you click on the "Recent Folders" tab does not update after you've opened or moved a message to a folder. It seems to only update by closing TB and reopening TB.

OK, I immediately ran a test with the debug option
extensions.quickfolders.debug.recentFolders = true

and getting the following output when clicking the recent folders tag / button:

QuickFolders {RECENTFOLDERS} 21:52:17.705  [38329 ms]	 
createRecentTab(  passedPopup: null, isDrag: false, passedButton: QuickFolders-Recent-CurrentFolderTool)
QuickFolders {RECENTFOLDERS} 21:52:17.705  [0 ms]	 
createRecentPopup(passedPopup:null, isDrag:false, isCreate:false)
QuickFolders {RECENTFOLDERS} 21:52:17.705  [0 ms]	 
Creating Popup Set for Recent Folders tab
QuickFolders {RECENTFOLDERS} 21:52:17.705  [0 ms]	 
generateMRUlist()
QuickFolders {RECENTFOLDERS} 21:52:17.718  [13 ms]	 
=============================
createRecentPopup Finished!

The code that is supposed to rebuild the most recent list is in generateMRUlist(), so I will debug that one in detail to see what the heck is happening... For now I am flagging this one as Tb78 regression, let's see what happens...

@swolh
Copy link
Author

swolh commented Nov 23, 2020 via email

@RealRaven2000
Copy link
Owner

RealRaven2000 commented Nov 23, 2020

Thank you

be careful if you reply to github issues via email - the complete text gets listed online - you may want to shorten it or answer directly on github - I can do it for you too if it's ok with you? I found the list is generated correctly but the old menupopup is not removed (in the original implementation it was the first child element of the button, so I guess we have duplicate menus and only the first one (outdated!) is being shown:

image

Should be relatively easy to fix, but a concern if I may have used the pattern (remove buttonElement.firstChild to get rid of an old popup) elsewhere.

@RealRaven2000
Copy link
Owner

...and here is a test version with a quick fix:

QuickFolders-wx-5.1pre2.zip

(download an drag to Thunderbird Add-ons manager to test - no restart necessary)

@RealRaven2000
Copy link
Owner

fixed in 5.1 released on 10/Dec/2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants