Skip to content

Conversation

@rlbxku1r
Copy link
Contributor

@rlbxku1r rlbxku1r commented Nov 26, 2025

Add Remove from favorites and Open containing folder action for favorites/recents, because you can't do this without using Nemo.

image

@mtwebster
Copy link
Member

I think this should be in the context menu (similar to what you get when right-clicking an application). This impacts the available width of the filename and path labels, and isn't consistent with the rest of the applet.

@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch from 29412e1 to cebbe7f Compare November 27, 2025 06:43
@rlbxku1r
Copy link
Contributor Author

rlbxku1r commented Nov 27, 2025

I think this should be in the context menu (similar to what you get when right-clicking an application). This impacts the available width of the filename and path labels, and isn't consistent with the rest of the applet.

Changed to the context menu action.

@rlbxku1r rlbxku1r changed the title menu: Add 'unfavorite document' button menu: Add 'unfavorite document' action Nov 27, 2025
@hthienloc
Copy link

The official menu applet should be Cinnamenu. It already has this feature and is far better than the default one.

image

@clefebvre clefebvre changed the title menu: Add 'unfavorite document' action [Next] menu: Add 'unfavorite document' action Nov 27, 2025
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch from 450dc24 to 88fb376 Compare November 27, 2025 11:48
@rlbxku1r rlbxku1r changed the title [Next] menu: Add 'unfavorite document' action [Next] menu: Add 'unfavorite document' and 'open containing folder' action Nov 27, 2025
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch from 88fb376 to 4adbf25 Compare November 27, 2025 12:32
@rlbxku1r rlbxku1r changed the title [Next] menu: Add 'unfavorite document' and 'open containing folder' action [Next] menu: Add context menu actions for favorites/recents Nov 27, 2025
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch 6 times, most recently from cec7133 to ba8b357 Compare November 30, 2025 06:12
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch 4 times, most recently from eb099fe to c3c8d19 Compare December 16, 2025 19:01
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch 3 times, most recently from 7c8ce87 to af0e5dd Compare December 27, 2025 14:26
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch from af0e5dd to 2f15996 Compare December 29, 2025 16:28
@clefebvre clefebvre changed the title [Next] menu: Add context menu actions for favorites/recents menu: Add context menu actions for favorites/recents Jan 22, 2026
Comment on lines 809 to 854
_openContainingFolder() {
try {
Gio.DBus.session.call_sync(
"org.freedesktop.FileManager1",
"/org/freedesktop/FileManager1",
"org.freedesktop.FileManager1",
"ShowItems",
new GLib.Variant("(ass)", [
[this._button.uri],
global.get_pid().toString()
]),
null,
Gio.DBusCallFlags.NONE,
1000,
null
);
} catch (e) {
global.logError(`Could not open containing folder: ${e}`);
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice, but only if your default file manager supports the FileManager service.

You could add a fallback that uses Gio.app_info_launch_default_for_uri() on the parent uri (you can use Gio.File methods to reliably get the parent uri). This won't 'select' the file but will at least follow through with opening the containing folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch 4 times, most recently from fa43fe2 to dbf6f05 Compare February 6, 2026 20:48
Add `Remove from favorites` and `Open containing folder` action for favorites/recents.
@rlbxku1r rlbxku1r force-pushed the menu-unfavorite-button branch from dbf6f05 to da22e34 Compare February 9, 2026 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants