Community
Participate
Working Groups
If you press cancel on the preference page, I'm surprised that we still trigger a CSS theming event. Maybe this can be avoided?
Andrew something for you?
+1 I've noticed this and it's annoying as a user. I'll have a look :)
I looked into the git history, and it seems that the cancel button was made to trigger the CSS engine in Bug 317125. I couldn't find an explanation as to why this was done. Personally, I see no reason to keep any theming logic in the cancel button (it should just close the preference dialog). If there's no objection to this, we can merge a patch to remove this theming logic.
+1
New Gerrit change created: https://git.eclipse.org/r/163840
(In reply to Andrew Obuchowicz from comment #3) > I looked into the git history, and it seems that the cancel button was made > to trigger the CSS engine in Bug 317125. > > I couldn't find an explanation as to why this was done. Personally, I see no > reason to keep any theming logic in the cancel button (it should just close > the preference dialog). > > If there's no objection to this, we can merge a patch to remove this theming > logic. I had to use theming preferences quite often recently (mostly involuntary) and noticed immediate changes in rendering from the "Color and Font Theme" drop down without pressing apply. Maybe this call on cancel was to revert such changes.
> I had to use theming preferences quite often recently (mostly involuntary) > and noticed immediate changes in rendering from the "Color and Font Theme" > drop down without pressing apply. Maybe this call on cancel was to revert > such changes. I think you solved the mystery, Paul :D In this case, this patch should also deal with preventing "eager" changes to the css engine. IMO changes should only be applied when the Apply or Apply and Close buttons are preseed.
(In reply to Andrew Obuchowicz from comment #7) > IMO changes should only be applied when the Apply or Apply > and Close buttons are preseed. I agree on that part that the drop-down is misbehaving.
(In reply to Paul Pazderski from comment #8) > (In reply to Andrew Obuchowicz from comment #7) > > IMO changes should only be applied when the Apply or Apply > > and Close buttons are preseed. > > I agree on that part that the drop-down is misbehaving. The latest gerrit fixes this :)