Color for checkerboard pattern background not remembered

[XnTriq]

The value for Alpha channel in Tools » Settings... » View » Use checkerboard pattern as background isn't remembered.

My settings:

• Background = R128/G128/B128/A255
  C'Pattern #1 = R255/G255/B255/A0
  C'Pattern #2 = R255/G255/B255/A51

[Clo]

XnTriq

Hello W. !

• In my xnview.ini file I've :

…
[Viewer]
…
useChecker=true
checkerColor1=255 255 255
checkerColor2=255 202 255
checkBoardSize=8
…

and the pink¦white pattern is kept alright here (for ages, I always install the new versions in the same dir…).

KR
Claude
Clo

[XnTriq]

Hi Clo =)

It's MP v0.38.7 that I'm talking about.

Code: Select all

[Viewer]
backColor=128 128 128
useChecker=true
checkerColor1=255 255 255
checkerColor2=255 255 255
checkBoardSize=8

[Clo]

XnTriq

Again…

• It's that I've in 0.38.7 too…
- Or do you mean, in other words, that CHANGING these colours is not saved in this version ? 

KR
Claude
Clo

[XnTriq]

It's that I've in 0.38.7 too…

Oh, sorry. Now I see...

Or do you mean, in other words, that CHANGING these colours is not saved in this version ?

Let me explain how to reproduce this bug step by step:

1. Download and open logo.png.
2. Go to Tools » Settings... » View.
3. Set the Background color to...
   • Red: 128
   • Green: 128
   • Blue: 128
   • Alpha channel: 255
4. Enable the Use checkerboard pattern as background option.
5. Change the first color to...
   • Red: 255
   • Green: 255
   • Blue: 255
   • Alpha channel: 0
6. Change the second color to...
   • Red: 255
   • Green: 255
   • Blue: 255
   • Alpha channel: 128
7. Leave the Options dialog by clicking OK to save the settings.
8. This is what I get:
   
9. Now change the Background color to...
   • Red: 0
   • Green: 255
   • Blue: 0
   • Alpha channel: 255
10. This is what I get:
    

So far, so good.
But when you enter the color picker for one of the checkerboard background colors again, the value for Alpha channel is 255 (it should be 0 and 128).

This is how %AppData%\XnViewMP\xnview.ini should look like:

[Viewer]
checkerColor1=255 255 255 0
checkerColor2=255 255 255 128

[Clo]

XnTriq

Again…

• I see now, however I think that the issue doesn't matter a lot finally since the “old” INI entries -that I'd set "by hand" in an old version- work perfectly and are kept through the versions…
- IMHO, the alpha channel is totally useless to define the colours of the chequered pattern… I think that it might be greyed out when the palette is used in this dalogue…
- What do you think ?

KR
Claude
Clo

[XnTriq]

I agree that the alpha channel is redundant (or even confusing) under most circumstances, except for the checkerbord background and Background color in Tools » Settings... » View » Info, because these two can act as (translucent) layers over other subjacent backgrounds.

We have infoOpacity for infoBackColor.
Maybe it's a good idea to remove the Alpha channel setting from the color picker all together and add a checkBoardOpacity1/checkBoardOpacity2 for checkerColor1/checkerColor2 instead?

[XnTriq]

In beta 0.38.8 and 0.38.9 the opacity is saved and remembered, ...

Code: Select all

checkerColor1=255 255 255 0
checkerColor2=255 255 255 51

... but when I go back to the color picker dialog, the value for Alpha channel is always 255.