b1: Filter option/page improvements

Bugs and Issues in 1.82 Test versions which have been fixed and verified as resolved.

Moderators: XnTriq, xnview

User avatar
Dreamer
XnThusiast
Posts: 4605
Joined: Sun Jul 25, 2004 9:08 pm
Location: Slovakia

b1: Filter option/page improvements

Post by Dreamer » Tue Nov 01, 2005 5:09 pm

Filter name should be renamed, since enabled = show (to File Handling?).

Extensions - remove "E.g. EPS..." and merge Extensions... and field [ . . . ] to 1 row.

Add fixed option "EPS..." files below Text files.

Completely remove "exclude list" - replace it with "Custom files"or even merge this option into one row:

[abc def ghi ] files . . . [x] [ ] [x] [x]

It would be clearer and same as other options, if someone wants to exclute filetype, he will unmark [x] to [ ].

Danny
Posts: 564
Joined: Sat Sep 04, 2004 5:09 pm

Post by Danny » Thu Nov 10, 2005 7:23 pm

Some thoughts of mine:

Seems like you forgot to remove the settings under "Browser -> File List" which are now obsolete (Items shown in ... mode).

In the browser's "Filter" settings it should be possible to disable "Files smaller than" by typing "0" into the text fields. Currently putting a "0" in there will show you no preview at all, which is useless since you can as well turn it off then.

It's not very clear that the "Exclude" checkbox and the text field below are related.

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

Re: Filter option/page improvements

Post by helmut » Thu Nov 10, 2005 9:20 pm

Dreamer wrote:Filter name should be renamed, since enabled = show (to File Handling?).
I agree. The options are the opposite of a Filter, it's the list of files shown I suggest to use "File List" for this category and rename the other one.
Dreamer wrote:Extensions - remove "E.g. EPS..." and merge Extensions... and field [ . . . ] to 1 row.
Not clear what you mean here.
Dreamer wrote:Add fixed option "EPS..." files below Text files.
Right.
Dreamer wrote:Completely remove "exclude list" - replace it with "Custom files"or even merge this option into one row:

[abc def ghi ] files . . . [x] [ ] [x] [x]

It would be clearer and same as other options, if someone wants to exclute filetype, he will unmark [x] to [ ].
The way the Custom extension are now is really confusing. It just says "Extensions (separated by a space)...)" and nothing else.

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

Post by helmut » Thu Nov 10, 2005 9:28 pm

Danny wrote:Some thoughts of mine:

Seems like you forgot to remove the settings under "Browser -> File List" which are now obsolete (Items shown in ... mode).
Exactly.
Danny wrote:In the browser's "Filter" settings it should be possible to disable "Files smaller than" by typing "0" into the text fields. Currently putting a "0" in there will show you no preview at all, which is useless since you can as well turn it off then.
Yes, "0" shouldn't be allowed.
Danny wrote:It's not very clear that the "Exclude" checkbox and the text field below are related.
Right. This has been heavily discussed in topic File Inclusion/Exclusion. The way it is now is really confusing.

Danny
Posts: 564
Joined: Sat Sep 04, 2004 5:09 pm

Post by Danny » Fri Nov 11, 2005 1:07 am

helmut wrote: Yes, "0" shouldn't be allowed.
Au contrair, mon frere. <- token french skills there... ;)

IMO "0" should be possible, but it should disable any kind of size limiting.

User avatar
Dreamer
XnThusiast
Posts: 4605
Joined: Sun Jul 25, 2004 9:08 pm
Location: Slovakia

Post by Dreamer » Fri Nov 11, 2005 1:28 am

Not clear what you mean here.
I mean - remove TEXT "E.g. EPS...", but this would be better and clearer:

[abc def ghi ] files . . . [x] [ ] [x] [x]

There could be some note below that there must be space between filetypes.
Danny wrote:IMO "0" should be possible, but it should disable any kind of size limiting.
I agree, it's a standard in other programs...

Minor - I don't like this " - " if option is not possible to set, I suggest appropriate greyed out checkbox [x] / [ ].

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

Post by helmut » Fri Nov 11, 2005 7:35 am

Dreamer wrote:
Not clear what you mean here.
I mean - remove TEXT "E.g. EPS...", but this would be better and clearer:

[abc def ghi ] files . . . [x] [ ] [x] [x]

There could be some note below that there must be space between filetypes.
I think that's a good idea. Having the edittext on the top and only have a note how to use this field at the bottom.
Danny wrote:IMO "0" should be possible, but it should disable any kind of size limiting.
I agree, it's a standard in other programs...[/quote]
No. We have another option for disabling, so we shouldn't misuse this option. Otherwise people will say: I have option to show preview checked, but it's not shown.
Dreamer wrote:Minor - I don't like this " - " if option is not possible to set, I suggest appropriate greyed out checkbox [x] / [ ].
I don't like the fat "-" either, it should be simply a bit thinner. Greyed-out checkboxes would be wrong, user will wonder: What do I have to do to make the non-grey and enabled? (Screen design rule: Never display a control if the user can never change it).

Danny
Posts: 564
Joined: Sat Sep 04, 2004 5:09 pm

Post by Danny » Fri Nov 11, 2005 12:39 pm

helmut wrote:
Danny wrote:IMO "0" should be possible, but it should disable any kind of size limiting.
I agree, it's a standard in other programs...
No. We have another option for disabling, so we shouldn't misuse this option. Otherwise people will say: I have option to show preview checked, but it's not shown.
You still don't understand. A "0" should disable the size limit so thumbnails and previews for files of all sizes will be shown. There's currently no setting to do this. All you can do is set a very big number in those fields like 99999.
Dreamer wrote:Minor - I don't like this " - " if option is not possible to set, I suggest appropriate greyed out checkbox [x] / [ ].
I don't like the fat "-" either, it should be simply a bit thinner. Greyed-out checkboxes would be wrong, user will wonder: What do I have to do to make the non-grey and enabled? (Screen design rule: Never display a control if the user can never change it).
Correct. Grayed-out means possible under certain circumstances.

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

Post by helmut » Fri Nov 11, 2005 5:24 pm

Danny wrote:... You still don't understand. A "0" should disable the size limit so thumbnails and previews for files of all sizes will be shown. There's currently no setting to do this. All you can do is set a very big number in those fields like 99999.
...
If you want to allow files with any size you can
... enter 99999
... clear the field

When entering "0", some people will expect that all files with be disallowed, other will expect that all files are allowed. An easy way to prevent misunderstandings is by simply by disallowing "0".

Danny
Posts: 564
Joined: Sat Sep 04, 2004 5:09 pm

Post by Danny » Fri Nov 11, 2005 6:38 pm

helmut wrote:
Danny wrote:... You still don't understand. A "0" should disable the size limit so thumbnails and previews for files of all sizes will be shown. There's currently no setting to do this. All you can do is set a very big number in those fields like 99999.
...
If you want to allow files with any size you can
... enter 99999
... clear the field

When entering "0", some people will expect that all files with be disallowed, other will expect that all files are allowed. An easy way to prevent misunderstandings is by simply by disallowing "0".
....

99999 would be 99MB. Far less than a lot of movie files. Have you actually tried clearing the field? Next time you open the program there's a "0" in there....

And what sense does it make to disallow any filesize this way? That's the same as disabling the preview function. But you can do that via "View" menu and you'll get more free window space on top of that -> far more usefull. So anyone expecting different hasn't put enough thought into it.

And as Dreamer already mentioned: most other programs (all that i know) work that way. 0 means disabled. (Azureus for example)

User avatar
Dreamer
XnThusiast
Posts: 4605
Joined: Sun Jul 25, 2004 9:08 pm
Location: Slovakia

Post by Dreamer » Sat Nov 12, 2005 12:25 am

helmut wrote:
Dreamer wrote:
Danny wrote:IMO "0" should be possible, but it should disable any kind of size limiting.
I agree, it's a standard in other programs...
No. We have another option for disabling, so we shouldn't misuse this option. Otherwise people will say: I have option to show preview checked, but it's not shown.
There could be a note below, we could use *, **...

BTW - I think similar option should be also for View / Fullscreen mode, now it's a bit confusing. If you agree, please start a new topic.

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

Post by helmut » Sat Nov 12, 2005 12:07 pm

Danny wrote:...
99999 would be 99MB. Far less than a lot of movie files.
Currently you can enter "9999999999999" which should be sufficient for most cases. (Personally I think that this limit is not so good since there are no thousand separators. Also for "No limit" another way might be found, see below).
Danny wrote:Have you actually tried clearing the field? Next time you open the program there's a "0" in there....
Currently, we have an Alpha version, so things may not work 100% properly. This is why we discuss here.
Danny wrote:And what sense does it make to disallow any filesize this way? That's the same as disabling the preview function. But you can do that via "View" menu and you'll get more free window space on top of that -> far more usefull. So anyone expecting different hasn't put enough thought into it.
Exactly. As written above the value "0" is not clear and for this reason it should not be allowed. This is why I suggested an error message when entering "0". In the error message there could be even a short explanation which values are valid.

A second way to make this clearer is by replacing "0" and the maximum value "9999999999999" with something more meaningful when leaving the field. E.g. the field could be cleared (empty), or a value "No limit", "Any", or "*" could be shown.

A third way is introducing a checkbox which enables/disables each limit (what we have in 1.80). But I think this would be confusing since we have many checkboxes on this category page, already.

A fourth way is dropdownlists: The dropdownlists could contain a value for "No limit" and meaningful, predefined limits. E.g.,

No limit
1 GB
100 MB
10 MB
1 MB
100KB
10 Kb
When choosing one of the predefined limits, the appropriate value in kilobytes would be entered in the field. Sure enough, apart from predefined values the user could enter arbitrary values.
Dreamer wrote:And as Dreamer already mentioned: most other programs (all that i know) work that way. 0 means disabled. (Azureus for example)
Hmm, that's an argument, but I don't think that this is a known and good standard. I guess in the programs you are refering to it says "Do not show files larger than". In this case, a "0" for all files makes sense and is logical since all files are larger than 0 KBytes. But in our case it says "Show files smaller than".

Personally I favour the "dropdownlists" approach. I think this would be clear regarding the value "No limit". And the predefined values also make things clearer and are convenient to use.

User avatar
Dreamer
XnThusiast
Posts: 4605
Joined: Sun Jul 25, 2004 9:08 pm
Location: Slovakia

Post by Dreamer » Sun Dec 11, 2005 11:59 pm

All fixed/changed except this:
Add fixed option "EPS..." files below Text files.

User avatar
Dreamer
XnThusiast
Posts: 4605
Joined: Sun Jul 25, 2004 9:08 pm
Location: Slovakia

Post by Dreamer » Mon Dec 19, 2005 11:16 pm

Dreamer wrote:All fixed/changed except this:
Add fixed option "EPS..." files below Text files.
Added, but why there is "AI" instead of "EPS"?

Also, now we have these filetypes twice, so I suggest to change text "e.g. "EPS PDF PS"" and default text below "EPS EPSF PS PDF" to e.g. EXE BAT COM...

User avatar
xnview
Author of XnView
Posts: 33644
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Post by xnview » Tue Dec 20, 2005 2:56 pm

Dreamer wrote:
Dreamer wrote:All fixed/changed except this:
Add fixed option "EPS..." files below Text files.
Added, but why there is "AI" instead of "EPS"?
EPS is like PS
Also, now we have these filetypes twice, so I suggest to change text "e.g. "EPS PDF PS"" and default text below "EPS EPSF PS PDF" to e.g. EXE BAT COM...
Right
Pierre.

Locked