Feature: Option to open all items in a tag #11598
Conversation
| @@ -62,6 +58,8 @@ public FileTagsWidgetViewModel ViewModel | |||
|
|
|||
| private ICommand OpenInNewPaneCommand; | |||
|
|
|||
| private ICommand OpenAllItems; | |||
There was a problem hiding this comment.
Maybe this could be moved to a rich command directly?
There was a problem hiding this comment.
@cinqmilleans can you check if I implemented the RichCommand properly?
|
|
||
| public string Label => "OpenAllItems".GetLocalizedResource(); | ||
|
|
||
| public RichGlyph Glyph { get; } = new("\uE8E5"); |
There was a problem hiding this comment.
It's a temp icon since OpacityIcon is not working
|
Was this added to the sidebar as well? |
Not yet, I need |
| @@ -48,6 +48,8 @@ public interface ICommandManager : IEnumerable<IRichCommand> | |||
|
|
|||
| IRichCommand RotateLeft { get; } | |||
| IRichCommand RotateRight { get; } | |||
|
|
|||
| IRichCommand OpenAllTaggedItems { get; } | |||
| if (sender is not Grid grid || grid.Parent is not Grid parent) | ||
| return; | ||
|
|
||
| LoadContextMenuItem(parent, e, true); |
There was a problem hiding this comment.
Would it be possible to create the menu in xaml?
There was a problem hiding this comment.
Yes, anyway I'll mark this as Draft and I'll change this once we have a final context, to avoid working twice
|
@ferrariofilippo what's the status of this PR? |
|
There are too many conflicts, I'll close this. |
|
@QuaintMako do you have any thoughts on a tags context? |
|
@QuaintMako can I start working on this using a |
Sorry for the very late answer, I don't see any issue creating a TagsContext. It might be helpful in the future for other actions surrounding the tags as well. |
Resolved / Related Issues
Items resolved / related issues by this PR.
Details
FileTagsWidget.xamlare needed to fix a bug where right clicking on the right of an item wouldn't show the context menuValidation
How did you test these changes?