Tags: Menu items

Bugs and Suggestions in XnView Classic which have been resolved

Moderators: XnTriq, xnview

Post Reply
User avatar
helmut
Posts: 8219
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Tags: Menu items

Post by 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).
User avatar
xnview
Author of XnView
Posts: 33836
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: Tags: Menu items

Post by xnview »

helmut wrote: 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!
Pierre.
User avatar
helmut
Posts: 8219
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: Tags: Menu items

Post by helmut »

xnview wrote:
helmut wrote: 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".
User avatar
Olivier_G
XnThusiast
Posts: 1423
Joined: Thu Dec 23, 2004 7:17 pm
Location: Paris, France
Contact:

Post by 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... :mrgreen:)
- 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
User avatar
Olivier_G
XnThusiast
Posts: 1423
Joined: Thu Dec 23, 2004 7:17 pm
Location: Paris, France
Contact:

Post by Olivier_G »

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

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
User avatar
Olivier_G
XnThusiast
Posts: 1423
Joined: Thu Dec 23, 2004 7:17 pm
Location: Paris, France
Contact:

Post by Olivier_G »

In beta 6, we have:

Code: Select all

Tag current
Untag current
Invert
Untag all
Show tag box
They work as expected... :D

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
Olivier
FlorianH
Posts: 121
Joined: Fri Dec 09, 2005 8:28 am
Location: Munich, Germany

Post by FlorianH »

Olivier_G wrote: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?
User avatar
helmut
Posts: 8219
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Post by helmut »

Olivier wrote: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.
marsh wrote: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
XnThusiast
Posts: 2443
Joined: Sun May 15, 2005 6:31 am

Post by marsh »

helmut wrote:
Olivier wrote: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.
marsh wrote: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. :twisted:
User avatar
helmut
Posts: 8219
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Post by helmut »

marsh wrote:Ha. It wasn't me. :twisted:
Oh, right, it was FlorianH.
Post Reply