1.3.1 - edit pallete - transparency | non existing index on hover

Bugs which are supposed to be fixed in the next test version (not available yet)

Moderators: XnTriq, helmut, xnview, Dreamer

user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

1.3.1 - edit pallete - transparency | non existing index on hover

Post 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

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
You do not have the required permissions to view the files attached to this post.
User avatar
xnview
Author of XnView
Posts: 39880
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

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

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

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

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.
You do not have the required permissions to view the files attached to this post.
User avatar
xnview
Author of XnView
Posts: 39880
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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
Last edited by user0 on Tue Feb 28, 2023 6:44 am, edited 1 time in total.
User avatar
xnview
Author of XnView
Posts: 39880
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

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

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post by xnview »

This problem is supposed to be fixed in XnView MP 1.4.3. Please check and confirm the bug fix here.
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

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

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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
  • 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
You do not have the required permissions to view the files attached to this post.
User avatar
xnview
Author of XnView
Posts: 39880
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.
Pierre.
user0
Posts: 386
Joined: Sat May 09, 2015 9:37 am

Re: 1.3.1 - edit pallete - transparency | non existing index on hover

Post 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.