1538520 - Enabling or disabling filters in the filter list doesn't stick, reverts to previous state when the panel is opened again

Status: RESOLVED FIXED

(Thunderbird :: Filters, defect)

Description

[Jorg K (CEST = GMT+2)]

Enabling or disabling filters in the filter list doesn't stick, reverts to previous state when the panel is opened again.

Comment 1

[:aceman]

Regressed by bug 1475817, https://hg.mozilla.org/comm-central/rev/c038f663b745 .

Comment 2

[:aceman]

Attachment: 1538520.patch

The main fix is the change from !this.checked to this.checked as the toggleFilter was storing the inverted value than was displayed.

I changed the listener from 'click' to 'CheckboxStateChange' event, because previously we listened to click and managed (in toggleFilter) what state was displayed for the checkbox. Now when it is a real checkbox element, it shows and manages its state by itself and we only follow and save the state into filter. So the state of the checkbox could change by other means, not just by click (thus onFilterClick wouldn't be called). So let's tie onFilterClick() to the checkbox state better.

Also notice that when you click the checkbox (or to the right of it), it toggles and the filter row loses proper selection. The background highlight turns grey from blue (on Linux). You can use up/down arrows to move the grey bar and then toggling checkboxes using space key produces interesting effects. I do not fix this here, but it is again a regression, it does not happen on TB60. Geoff, please file and fix that separately.

Comment 3

[Jorg K (CEST = GMT+2)]

This works OK in TB 66 beta 3, so bug 1475817 can't be the culprit here.

Comment 4

[:aceman]

Ok, then I guess bug 1455433.

Comment 5

[Geoff Lankow (:darktrojan)]

Comment on attachment 9053103 [details] [diff] [review]
1538520.patch

I don't think the change to `CheckboxStateChange` is necessary but it's not worth debating.

Comment 6

[:aceman]

I think otherwise the enabled state of the filter and the checkbox may diverge if e.g. someone sets checkbox.checked = value instead of clicking it.
So let's try the patch as is, thanks.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9938df594a75
make 'Enabled' filter property in filter list stick by properly checking the state of the checkbox. r=darktrojan

Comment 8

[Jorg K (CEST = GMT+2)]

TB 67 beta:
https://hg.mozilla.org/releases/comm-beta/rev/6360477e63b526c0babdf30bda7f4ddfc54e6748