ifarup / ciefunctions

Python module, GUI, and web app for computing the CIE TC1-97 colorimetric functions
GNU General Public License v3.0
5 stars 5 forks source link

All types of Plot-windows in Python GUI and web-app: Remove checkmarks in those check boxes that are not active ("greyed out"). #97

Closed janhenrikwold closed 9 years ago

janhenrikwold commented 10 years ago

Alternatively (if possible): In each plot window, remove the inactive check boxes (with accompanying text) completely.

ifarup commented 10 years ago

I would like to remind you that this is how it used to be in the python app in the beginning, but that we (you) suggested to use graying out instead due to the difficulty of finding a good way to arrange and rearrange the remaining check boxes in the gui. Maybe the web is more flexible in this respect.

janhenrikwold commented 10 years ago

Yes, I remember very well, and I must admit that I am a bit ambivalent about this.

What do you think Herr Farup? I think maybe "Greying out" will be OK . For a short moment yesterday, however, I was somewhat inclined for the other solution, but after having thought of it for a while i switched to the greying option again -- so is the life of Ångstrøm :-)

Anyway, I think "greying out" will suffice for now. (Some members attending the TC meeting might have some preferences.)

Keeping the boxes visible, I think the inactive boxes should always be unchecked.

ifarup commented 10 years ago

Personally, I still prefer the grey version. And I think that the check should also stay, so the user knows what to expect whenever a plot with the option enabled is reopened.

janhenrikwold commented 10 years ago

Well, to me it seems a bit strange with a the plot-window for (for instance) ‘MacLeod-Boynton ls diagram’ with the checkbox ‘Compare with CIE Standard 1964’ checked (though grey).

From: Ivar Farup [mailto:notifications@github.com] Sent: 16. juni 2014 04:30 To: ifarup/tc1-82 Cc: Jan Henrik Wold Subject: Re: [tc1-82] All types of Plot-windows in Python GUI and web-app: Remove checkmarks in those check boxes that are not active ("greyed out"). (#97)

Personally, I still prefer the grey version. And I think that the check should also stay, so the user knows what to expect whenever a plot with the option enabled is reopened.

— Reply to this email directly or view it on GitHubhttps://github.com/ifarup/tc1-82/issues/97#issuecomment-46152778.

gerardodiego commented 10 years ago

Same for me. Gray is a font color. If I choose white instead, they'll be hidden.

ifarup commented 9 years ago

Split into #113 and #114