b3: [minor] controls not disabled in options

FlorianH · Post by **FlorianH** » Fri Jan 06, 2006 10:23 am

1) go to options - slideshow
2) disable 'show information' and close options with 'ok' button
3) reopen options - slideshow
-> button 'text settings...' is enabled but should be disabled.

Similiar behaviour for 'enable caching' in options - browser - cache, where directory textbox and '...' button is not disabled if caching is disabled...

XnView 1.82 -

Xyzzy · Post by **Xyzzy** » Fri Jan 06, 2006 10:30 am

I do not agree. You should be able to edit information regardless of if it is displayed.

X.

FlorianH · Post by **FlorianH** » Fri Jan 06, 2006 10:41 am

Xyzzy wrote:I do not agree. You should be able to edit information regardless of if it is displayed.

a) You can't edit information. You can only edit text settings (font, font size, etc.) and only after you reopen the options dialog.
b) It's not usefull to change cache directory if cache is disabled.

Xyzzy · Post by **Xyzzy** » Fri Jan 06, 2006 10:52 am

OK, I see that Slide Show behaviour is inconsistent, it should be fixed(however I think editing should be possible if Show information is unchecked).

But ... for cache can stay active. It makes sense changing caching directory even if caching is disabled- why restrict it?

X.

FlorianH · Post by **FlorianH** » Fri Jan 06, 2006 5:41 pm

Xyzzy wrote:But ... for cache can stay active. It makes sense changing caching directory even if caching is disabled- why restrict it?

X.

Hm, i'm not realy convinced... But it's not really this important...

- Another error with the textbox is, that it doesn't display the cache directory, if it's set to the default path in %%xnview-install-dir%%. So user doesn't know where cache is...
- And perhaps renaming the label 'Directory' to 'Cache location' would be more clearly.

BTW: If cache directory is moved, shouldn't there be a question if cache db files should be moved too (or deleted)?

FlorianH · Post by **FlorianH** » Tue Jan 10, 2006 7:45 pm

RC1:
- controls disabled correctly
- textbox still doesn't display the cache directory, if it's set to the default path in %%xnview-install-dir%% -> is there any reason?

Post by **xnview** » Tue Jan 10, 2006 8:11 pm

FlorianH wrote:RC1:
- controls disabled correctly
- textbox still doesn't display the cache directory, if it's set to the default path in %%xnview-install-dir%% -> is there any reason?

yes because it's the default

Xyzzy · Post by **Xyzzy** » Thu Jan 12, 2006 9:10 am

Empty box is OK.

If we put there some info like <default>, user still can wonder where it is.
If we put there some pseudo-variable, see above and it could confuse user that XnView uses such variables.
If we put some fixed directory there, there is a problem with moving ini.
If we put dynamically some directory there, it may confuse user because we have a setting that changes dynamically without user interaction.

X.

FlorianH · Post by **FlorianH** » Thu Jan 12, 2006 7:36 pm

Xyzzy wrote:Empty box is OK.
...
If we put some fixed directory there, there is a problem with moving ini.
...

Oh yes, that's an argument i didn't think of so far. But why don't we write a relative path ("./cache") if the cache is in a subdir of the application dir?

Xyzzy · Post by **Xyzzy** » Thu Jan 12, 2006 10:05 pm

OK, that could do. Especially that this would require (I think) just a small change in default ini file.

Support.

Reopen and we'll see.

X.