Closed Bug 1762895 Opened 3 years ago Closed 3 years ago

With multiple recipient pills selected, focusing other spots in composition often fails to deselect all of them (e.g. `End`, `Ctrl+Shift+T/C/B`, after adding attachment etc.)

Categories

(Thunderbird :: Message Compose Window, defect)

defect

Tracking

(thunderbird_esr91 wontfix)

RESOLVED FIXED
101 Branch
Tracking Status
thunderbird_esr91 --- wontfix

People

(Reporter: thomas8, Assigned: thomas8)

References

(Regression)

Details

(Keywords: access, regression, ux-mode-error)

Attachments

(1 file)

STR

  1. Select 3 pills in To field of composition, focus on pill 3 (can use Shift+ArrowRight)
  2. Press End

Actual

  • only the focused pill 3 gets deselected
  • 2 pills remain selected
  • pretty irritating for users who depend on keyboard access and happen to use this method

Expected

  • All pills deselected

I have a fix.

Summary: With several recipient pills selected, using `End` key to focus address input fails to deselect all of them → With multiple recipient pills selected, using `End` key to focus address input fails to deselect all of them

More failure cases discovered which should deselect all pills, but only deselect focus pill, here's the full list.
Select multiple pills, then...

  • press End to focus address input
  • press Ctrl+Shift+C when selected pills are in To (and Ctrl+Shift+T/B by analogy), which focuses Cc row
  • choose another sender, which focuses the Fromselector
  • add an attachment, which moves the focus to the attachment
Summary: With multiple recipient pills selected, using `End` key to focus address input fails to deselect all of them → With multiple recipient pills selected, focusing other spots in composition often fails to deselect all of them (e.g. `End`, `Ctrl+Shift+T/C/B`, after adding attachment etc.)

When some other element outside of pills gets focused, make sure to deselect all
pills in the addressing area, not just the focused pill.

Attachment #9270867 - Attachment description: Bug 1762895 - Make blur event on a pill deselect *all* pills if relatedTarget is not a pill or null. r=aleca → Bug 1762895 - Make blur event on a pill deselect *all* pills if relatedTarget is not a pill or null. r=henry
Target Milestone: --- → 101 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/4bb056188757
Make blur event on a pill deselect all pills if relatedTarget is not a pill or null. r=henry

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: