[Terminal] remember color schema settings [easy]

[Disreali]

Using hrev43534, but it has been around for quite awhile.

Steps to reproduce:

1. Open Terminal -> Settings -> Settings.

• notice that the Color schema dropdown box is set to 'Custom', and the Color dropdown and ColorChooser(is that the proper name?) are 'live' and can be changed.

2. Select a pre-defined Color schema from the dropdown box.

• notice that the Color dropdown and the ColorChooser are now grayed out and no longer 'live'.

3. Click the 'OK' button to save settings and close the Settings window.

4. Repeat step 1

• Notice that the Color schema dropdown is once again set to 'Custom', and that the Color dropdown and ColorChooser are again 'live' and not grayed out.

Once a user has selected a pre-defined color schema, the settings window should remember it and show it as selected every time the settings are opened.

[siarzhuk]

Are you using non-English locale?

[Disreali]

Replying to siarzhuk:

Are you using non-English locale?

No. The system locale is en-US. I did not know that made a difference.

[diver]

Could be related to #7209.

[siarzhuk]

Replying to luckypower:

GSOC2012

???

[siarzhuk]

Replying to siarzhuk:

GSOC2012

???

Ups. :-( All is OK. Thank for the submission, luckypower!

Sorry for the noise!

[luckypower]

Replying to siarzhuk:

Replying to siarzhuk:

GSOC2012

???

Ups. :-( All is OK. Thank for the submission, luckypower!

Sorry for the noise!

You are welcome :)

[bonefish]

Any explanation for the patch? I haven't looked at the related code, but just looking at the patch I wonder why a == operator should not compare certain attributes. Intuitively I'd say that's the wrong place for a fix.

[siarzhuk]

Replying to bonefish:

Any explanation for the patch? I haven't looked at the related code, but just looking at the patch I wonder why a == operator should not compare certain attributes. Intuitively I'd say that's the wrong place for a fix.

I suspect that both cursor colors of the Color object are not used at the moment. So if the Color objects really contain not-initialized trash here - it is not a wonder why user see the "Custom" scheme every time he tries to set the pre-defined scheme. So the patch can be related to the real source of problem and can be counted as valual for GSoC. From the other side I have on my TODO list many tasks related to Terminal problems and restoring normal colors support is one of them I'm dreaming all this years when my MC goes to acid "green instead white" color scheme. ;-) Thats why I do not applied this.

[bonefish]

Replying to siarzhuk:

I suspect that both cursor colors of the Color object are not used at the moment. So if the Color objects really contain not-initialized trash here

I would suspect something like that as well, but if that's the case, the better solution would be either to remove these attributes completely (not just in the == operator) or -- particularly if there's any intention to use them in the future -- initialize them properly (and keep the comparison in the == operator).

[siarzhuk]

Replying to bonefish:

Replying to siarzhuk:

I suspect that both cursor colors of the Color object are not used at the moment. So if the Color objects really contain not-initialized trash here

I would suspect something like that as well, but if that's the case, the better solution would be either to remove these attributes completely (not just in the == operator) or -- particularly if there's any intention to use them in the future -- initialize them properly (and keep the comparison in the == operator).

You are right. Correct initialization of the object fileds is looking like more correct way to solve theproblem at the moment. ;-)

[luckypower]

Replying to siarzhuk:

Replying to bonefish:

Replying to siarzhuk:

I suspect that both cursor colors of the Color object are not used at the moment. So if the Color objects really contain not-initialized trash here

I would suspect something like that as well, but if that's the case, the better solution would be either to remove these attributes completely (not just in the == operator) or -- particularly if there's any intention to use them in the future -- initialize them properly (and keep the comparison in the == operator).

You are right. Correct initialization of the object fileds is looking like more correct way to solve theproblem at the moment. ;-)

Thanks for correcting me. cursor_fore_color and cursor_back_color was not set when we change the schema. So i thought it must not be checked when we compare schema. But I will come up with better solution, as both of you suggested.

[kallisti5]

solved in hrev44955

[siarzhuk]

Replying to kallisti5:

solved in hrev44955

On non-English locale it doesn't looks like "really" solved at the moment. ;-) Colors customization is broken in such circumstances.

[siarzhuk]

Fixed in hrev45266.