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
test_file.png (186 Bytes) Viewed 3263 times

Issue A
When 8-color file saved and reopened it shows more colors. Why?
a.png

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.
b.png

Issue C
There is only 8 colors in image.
However it shows all other indexes when hover mouse over this area.
c.png

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.
suggestion.png

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 :D

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 :D

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
how_to_set_trnsparency.png

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 A - fixed
  • 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 C - fixed
  • 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]
:bugconfirmed: 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
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
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 :shock:
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.