Page 1 of 2
1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Fri Nov 18, 2022 3:05 pm
by user0
XnView: MP 1.3.1 64bits
OS: Windows 64bit (22H2), display resolution 3840x2160, win scaling 150%
Edit colormap/pallete - not sure I fully understand how it works.
I have 8-colors file (no alpha) with 3 colors:

- test_file.png (186 Bytes) Viewed 3263 times
Issue A
When 8-color file saved and reopened it shows more colors. Why?
Issue B
There are no any transparent colors in this file.
Why Transparency checkbox is always ticked by default for any color you choose?
It shall be ticked only if color set to transparent.
Issue C
There is only 8 colors in image.
However it shows all other indexes when hover mouse over this area.
Suggestion
There is no need to duplicate Index/RGB information.
Leave only one instance, by default it shows details of the selected color.
However on_hover it shows details of the color under cursor. Once there are no any color under cursor - show details of selected color OR blank if no selected.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Sun Dec 04, 2022 2:05 pm
by xnview
user0 wrote: Fri Nov 18, 2022 3:05 pm
Issue A
When 8-color file saved and reopened it shows more colors. Why?
XnView always save with 16 colors colormap, 8 colors are not supported
Issue B
Issue C
Right
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Feb 01, 2023 2:05 pm
by xnview
user0 wrote: Fri Nov 18, 2022 3:05 pm
Issue B
If you click on an index, you want to use it as transparency...
Issue C
There is only 8 colors in image.
However it shows all other indexes when hover mouse over this area.
See
issue for current status and some details.
Leave only one instance, by default it shows details of the selected color.
However on_hover it shows details of the color under cursor. Once there are no any color under cursor - show details of selected color OR blank if no selected.
See
issue for current status and some details.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Feb 01, 2023 3:14 pm
by user0
xnview wrote: Wed Feb 01, 2023 2:05 pm
user0 wrote: Fri Nov 18, 2022 3:05 pm
Issue B
If you click on an index, you want to use it as transparency...
Not sure I got your point
The problem - transparency checkbox is
always ticked, no matter if index is set to transparent or not.
- Current behaviour:
if you want to set index to transparent you need to untick checkbox and than tick it again. Very weird.
- Expected behaviour:
Transparency checkbox shall be ticked only if the index you selected is set to transparent, if not - checkbox unticked.
ps: just noticed few more issues:
1.4.2 - edit pallete - applying transparency inconvenience
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Thu Feb 02, 2023 9:35 am
by xnview
user0 wrote: Wed Feb 01, 2023 3:14 pm
The problem - transparency checkbox is
always ticked, no matter if index is set to transparent or not.
- Current behaviour:
if you want to set index to transparent you need to untick checkbox and than tick it again. Very weird.
- Expected behaviour:
Transparency checkbox shall be ticked only if the index you selected is set to transparent, if not - checkbox unticked.
I don't understand. When you click on an index, the checkbox is checked. No need to uncheck it before.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Thu Feb 02, 2023 1:34 pm
by user0
xnview wrote: Thu Feb 02, 2023 9:35 am
user0 wrote: Wed Feb 01, 2023 3:14 pm
The problem - transparency checkbox is
always ticked, no matter if index is set to transparent or not.
- Current behaviour:
if you want to set index to transparent you need to untick checkbox and than tick it again. Very weird.
- Expected behaviour:
Transparency checkbox shall be ticked only if the index you selected is set to transparent, if not - checkbox unticked.
I don't understand. When you click on an index, the checkbox is checked. No need to uncheck it before.
lets go the other way
use case:
I want to make red color transparent in the attached test file
steps to achieve:
1. open test file
2. goto Edit Pallete
3. select red color (index 1)
Transparency checkbox is ticked. Why is the index not transparent?
4. untick Transparency checkbox
5. tick Transparency checkbox
6. press OK
7. press CTRL+S to save image
8. press CTRL+R to reload image
finally have transparency effect on red color
ps: maybe initial idea of Transparency checkbox just to show if index support Transparency, but not to represent its state?
Are there cases within this dialog when index does not support transparency? If yes, there shall be 2 checkboxes: support transparency and transparency state.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Fri Feb 03, 2023 9:29 am
by xnview
user0 wrote: Thu Feb 02, 2023 1:34 pm
steps to achieve:
1. open test file
2. goto Edit Pallete
3. select red color (index 1)
Transparency checkbox is ticked. Why is the index not transparent?
index 1 is transparent, if you press Ok, you can see that
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Fri Feb 03, 2023 10:42 am
by user0
xnview wrote: Fri Feb 03, 2023 9:29 am
user0 wrote: Thu Feb 02, 2023 1:34 pm
steps to achieve:
1. open test file
2. goto Edit Pallete
3. select red color (index 1)
Transparency checkbox is ticked. Why is the index not transparent?
index 1 is transparent, if you press Ok, you can see that
Issue D
ah.. this is another problem due to windows scale <>100% (I have 150%)
in my case with default xnview settings, after step 3, red color still remains visible,
but if i zoom-in/out it becomes transparent
when I change 'High zoom quality' to Never in Settings - transparency effect visible right after pressing OK button in 'Edit pallete' dialog
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Fri Feb 03, 2023 12:48 pm
by xnview
user0 wrote: Fri Feb 03, 2023 10:42 am
xnview wrote: Fri Feb 03, 2023 9:29 am
user0 wrote: Thu Feb 02, 2023 1:34 pm
steps to achieve:
1. open test file
2. goto Edit Pallete
3. select red color (index 1)
Transparency checkbox is ticked. Why is the index not transparent?
index 1 is transparent, if you press Ok, you can see that
ah.. this is another problem due to windows scale <>100% (I have 150%)
in my case with default xnview settings, after step 3, red color still remains visible,
but if i zoom-in/out it becomes transparent
when I change 'High zoom quality' to Never in Settings - transparency effect visible right after pressing OK button in 'Edit pallete' dialog
this one was well hidden

See
issue for current status and some details.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Mon Feb 27, 2023 10:03 am
by xnview
This problem is supposed to be fixed in
XnView MP 1.4.3. Please check and confirm the bug fix here.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Tue Feb 28, 2023 6:54 am
by user0
1.4.3
- issue B - unchanged. I still think Transparency checkbox should not be always ticked by default, but represent index's real state (ticked if index is transparent, unticked if index is not transparent)
- issue D - fixed, but there is another issue
- dirty_file_indicator (*) does not appear on image's tab once you modify index (change color/transparency), and
- you can goto prev/next file without any warnings, your "changed" image will remain it's changes until you reopen it or go back and press ctrl+r to reveal its real condition
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Mar 01, 2023 8:27 am
by xnview
user0 wrote: Tue Feb 28, 2023 6:54 am
- issue B - unchanged. I still think Transparency checkbox should not be always ticked by default, but represent index's real state (ticked if index is transparent, unticked if index is not transparent)
how that??
- dirty_file_indicator (*) does not appear on image's tab once you modify index (change color/transparency), and
- you can goto prev/next file without any warnings, your "changed" image will remain it's changes until you reopen it or go back and press ctrl+r to reveal its real condition
[/list]

Thanks to your detailed description I can reproduce
the problem.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Mar 01, 2023 11:11 am
by user0
xnview wrote: Wed Mar 01, 2023 8:27 am
user0 wrote: Tue Feb 28, 2023 6:54 am
- issue B - unchanged. I still think Transparency checkbox should not be always ticked by default, but represent index's real state (ticked if index is transparent, unticked if index is not transparent)
how that??
image has one transparent color - green, index 0

- test_file_transparency_checkbox.png (199 Bytes) Viewed 2447 times
- When I click on index 0 - Transparency checkbox is ticked (as expected)
- When I click on any other index (which is not transparent), Transparency checkbox is automatically checked.
Why?
It should be checked only for the index which is transparent.
If user wants to make any other index transparent - user shall manually tick this checkbox.

- transparency_checkbox.png (8.86 KiB) Viewed 2447 times
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Mar 01, 2023 2:34 pm
by xnview
user0 wrote: Wed Mar 01, 2023 11:11 am
Why?
It should be checked only for the index which is transparent.
If user wants to make any other index transparent - user shall manually tick this checkbox.[/list]
Oh ok. The transparent index has the border, so when you click on an index, the transparent index changes.
Re: 1.3.1 - edit pallete - transparency | non existing index on hover
Posted: Wed Mar 01, 2023 3:03 pm
by user0
xnview wrote: Wed Mar 01, 2023 2:34 pm
The transparent index has the border, so when you click on an index, the transparent index changes.
ohhh, I see what you have done here, very confusing

I always thought this border is just an index selection indicator as it behaves as such.
The issue is that 'Edit Palette' dialog is not only to set Transparency, but to modify colors as well.
Currenct behaviour adds extra step - unticking checkbox if all you want is just change colors.