Closed Bug 481682 Opened 16 years ago Closed 13 years ago

Thunderbird Message Filters enabled with spacebar won't save enabled/disabled state on close of filter list window

Categories

(MailNews Core :: Filters, defect)

x86
Windows XP
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 11.0

People

(Reporter: baffoni, Assigned: aceman)

References

Details

(Keywords: regression)

Attachments

(1 file)

Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9.1b4pre) Gecko/20090305 Shredder/3.0b3pre 1) Create or identify mail filter under tools/mail filters 2) Use mouse to toggle mail filter status (e.g. enabled to disabled) 3) Close filter window 4) Open filter window - mail filter is toggled (e.g. disabled). 5) Use space bar to toggle status (e.g. disabled to enabled). 6) Closed Mail filter window 7) Open Mail filter window Expected behavior: 8) Mail filter is toggled (e.g. enabled). Actual behavior: 8) Mail filter is not toggled (e.g. disabled).
Michael, is this a regression from version 2? sev > minor because there is a workaround (surprised you missed that) note: the first filter selected requires two spacebars to toggle Enable checkbox
Severity: normal → minor
Component: Preferences → Filters
Product: Thunderbird → MailNews Core
QA Contact: preferences → filters
I don't know if it is a regression in that I've never seen this error occur in TB2, it just worked normally in TB2. Two spacebars did not fix this (it just removes the check). One space toggles and a check appears, but it doesn't save it. I don't know why a second spacebar, which would remove the check, would be correct or intuitive as a solution, or why this would be a work around (it isn't, when the check is removed, or enabled, it doesn't change state if it is done with the space bar). No matter what state the filter is in, no amount of even or odd spacebar taps will change the state after closing and reopening.
the workaround is click. didn't mean to imply second spacebar works, as you found. whether it fails in v2 is a nit but figured I'd ask because you probably wouldn't have noticed it if you didn't already know it worked in v2. Your statement indicates it did, so this would be considered a regression. So adding appropriate keyword
Keywords: regression
Attached patch trivial patch (deleted) — Splinter Review
This fixes it for me. Can it be this simple?
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #580506 - Flags: review?(dbienvenu)
Summary: Thunderbrid 3 Message Filters enable with spacebar won't save on close of window → Thunderbird Message Filters enabled with spacebar won't save enabled/disabled state on close of filter list window
Comment on attachment 580506 [details] [diff] [review] trivial patch thx, seems to work fine.
Attachment #580506 - Flags: review?(dbienvenu) → review+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
Blocks: 388191
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: