Tags: Menu items

[helmut]

The new tags allow for marking/unmarking files in the browser.

- Menu structure
In Alpha1 the menu structure in "View > Tags" menu is:

Code: Select all

Tag current
Untag all
Show tag box

Instead, I suggest:

Code: Select all

Show
-------------------
Tag               T
Untag             T
-------------------
Tag all
Untag all
Invert all

Notes:
- "Show" is at top, although it might not be used that often. Reason is same menu structure like "View > Explorer pane >" and "View > Preview pane".
- "Tag all" and "Invert all" might be useful in some cases.

Same shortcut for tagging and untagging
The short cut "T" is good (does not work at the moment), later it should allow for tagging and untagging. This might be a bit tricky if multiple files are selected and the user might need to press "T" twice, but still faster than using the mouse and menu.

"Tag all" and "Untag all" needed?
I even wonder whether "Tag all" and "Untag all" are needed. This could be quickly achieved by using "Strg+A" (select all) and then "T" (once for tagging and twice for untagging).

[xnview]

Tag all
Untag all
Invert all

I'm not sure that tag/untag all is needed, because if we use 'Select All', it's the same!

[helmut]

Tag all
Untag all
Invert all

I'm not sure that tag/untag all is needed, because if we use 'Select All', it's the same!

Yes, I have noticed. When selecting several files and then "Tag", all selected files are tagged (this is even another reason why "Tag current" is wrong). Not quite sure, but I think you should get rid of both "Tag all" and "Untag all".

[Olivier_G]

What about?

Tag selected
Untag selected
Invert

- 'Show tags' should probably be moved to 'Options' (yes: it's me, the "Options killer man" who is saying this... )
- Tag/Untag should have different shortcuts, as one could make a selection with already tagged files and want to quickly Tag or Untag the whole selection without having to make several selections.
- A global "Invert tag" is enough (no need to add a "Invert tag within selection"... even if it may find some usage)

[Olivier_G]

'Tag current' actually invert tag status of all selected files...

I think Tagging and Untagging should be 2 different actions. For example, I tag several files and see afterwards I want to remove continuous series (some tagged, some not): I just need to select those series and Untag them without losing other tags (with simple Invert, I can't do that easily).

So what about:

Code: Select all

Show
------
Tag
Untag
Invert

(using current shortcut for Invert would actually be good. Maybe no need to have specific shorcuts for Tag & Untag)

[Olivier_G]

In beta 6, we have:

Code: Select all

Tag current
Untag current
Invert
Untag all
Show tag box

They work as expected...

Some comments:
- I prefer *nothing* to "current" (it works for selected files).
- "Invert tag" might be better than just "Invert"...
- 'Untag all' is not needed IMO (select all+Untag or just change folder... as it's not permanent).
- "Show tag box" at the end may be a good choice. But I would prefer a separation line from other tag actions.

For shortcuts, currently:
- [space] = Invert
- T = Tag
- Ctrl+T = Tag
I believe only 'Invert' should have a shortcut because it's the most straightforward action (tag/untag similarly) and covers 95% of all situations.

More opinions?
-> moved to 'Feedback'

Edit: So my updated choice would be:

Code: Select all

Tag
Untag
Invert tag     [T]
-----------
Show tag box

[FlorianH]

Edit: So my updated choice would be:

Code: Select all

Tag
Untag
Invert tag     [T]
-----------
Show tag box

Much better!

BTW: 'show tag box' should be enabled only in tumbnail-views, or?

[helmut]

Edit: So my updated choice would be:

Code: Select all

Tag
Untag
Invert tag     [T]
-----------
Show tag box

Olivier's suggestion is now implemented in RC1. -> Closed.

BTW: 'show tag box' should be enabled only in tumbnail-views, or?

A minor issue and I'm not so sure whether it really should be disabled. Please write a separate topic, marsh.

[marsh]

Edit: So my updated choice would be:

Code: Select all

Tag
Untag
Invert tag     [T]
-----------
Show tag box

Olivier's suggestion is now implemented in RC1. -> Closed.

BTW: 'show tag box' should be enabled only in tumbnail-views, or?

A minor issue and I'm not so sure whether it really should be disabled. Please write a separate topic, marsh.

Ha. It wasn't me.

[helmut]

Ha. It wasn't me.

Oh, right, it was FlorianH.