pax/booru-viewer
1
0
Fork 0
Code Issues Releases 18 Activity

context_menus: make Save/Unsave from Library mutually exclusive

Browse Source 
Previously both Save to Library submenu and Unsave from Library
showed simultaneously for saved posts. Now only the relevant action
appears based on whether the post is already in the library.

Also removed stale _current_post override on unsave — get_preview_post
already resolves the right-clicked post via grid selection index.

behavior change: browse grid context menu shows either Save or
Unsave, never both.
This commit is contained in:
pax 2026-04-11 22:13:21 -05:00
parent 79419794f6
commit 4bcff35708
1 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
booru_viewer/gui/context_menus.py
View File

@ -37,19 +37,22 @@ class ContextMenuHandler:
save_as = menu.addAction("Save As...") save_as = menu.addAction("Save As...")
from ..core.config import library_folders from ..core.config import library_folders
save_lib_menu = None
save_lib_unsorted = None
save_lib_new = None
save_lib_folders = {}
unsave_lib = None
if self._app._post_actions.is_post_saved(post.id):
unsave_lib = menu.addAction("Unsave from Library")
else:
save_lib_menu = menu.addMenu("Save to Library") save_lib_menu = menu.addMenu("Save to Library")
save_lib_unsorted = save_lib_menu.addAction("Unfiled") save_lib_unsorted = save_lib_menu.addAction("Unfiled")
save_lib_menu.addSeparator() save_lib_menu.addSeparator()
save_lib_folders = {}
for folder in library_folders(): for folder in library_folders():
a = save_lib_menu.addAction(folder) a = save_lib_menu.addAction(folder)
save_lib_folders[id(a)] = folder save_lib_folders[id(a)] = folder
save_lib_menu.addSeparator() save_lib_menu.addSeparator()
save_lib_new = save_lib_menu.addAction("+ New Folder...") save_lib_new = save_lib_menu.addAction("+ New Folder...")
unsave_lib = None
if self._app._post_actions.is_post_saved(post.id):
unsave_lib = menu.addAction("Unsave from Library")
copy_clipboard = menu.addAction("Copy File to Clipboard") copy_clipboard = menu.addAction("Copy File to Clipboard")
copy_url = menu.addAction("Copy Image URL") copy_url = menu.addAction("Copy Image URL")
copy_tags = menu.addAction("Copy Tags") copy_tags = menu.addAction("Copy Tags")
@ -108,7 +111,6 @@ class ContextMenuHandler:
elif id(action) in save_lib_folders: elif id(action) in save_lib_folders:
self._app._post_actions.save_to_library(post, save_lib_folders[id(action)]) self._app._post_actions.save_to_library(post, save_lib_folders[id(action)])
elif action == unsave_lib: elif action == unsave_lib:
self._app._preview._current_post = post
self._app._post_actions.unsave_from_preview() self._app._post_actions.unsave_from_preview()
elif action == copy_clipboard: elif action == copy_clipboard:
self._app._copy_file_to_clipboard() self._app._copy_file_to_clipboard()