Closed Bug 1868094 Opened 2 years ago Closed 2 years ago

When navigating by keyboard in the folder pane, the selected row should always be visible

Categories

(Thunderbird :: Folder and Message Lists, defect)

Thunderbird 115
Desktop
Unspecified
defect

Tracking

(thunderbird_esr115 affected, thunderbird121 wontfix, thunderbird123 fixed, thunderbird124 verified)

RESOLVED FIXED
123 Branch
Tracking Status
thunderbird_esr115 --- affected
thunderbird121 --- wontfix
thunderbird123 --- fixed
thunderbird124 --- verified

People

(Reporter: welpy-cw, Assigned: freaktechnik)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

Steps to reproduce:

  1. Make sure the folder pane header is shown and that the folder pane splitter's scrollbar is accessible.
  2. Press [End] to go to the last row.
  3. Use the up arrow key multiple times to go up.

Actual results:

When navigating beyond the top visible row, the newly selected row stays hidden behind the folder pane header.

Expected result:

The selected row should be scrolled into view.

Assignee: nobody → h.w.forms
Status: NEW → ASSIGNED
See Also: → 1863680
Attachment #9366802 - Attachment is obsolete: true
Assignee: h.w.forms → nobody
Status: ASSIGNED → NEW

Just to bring this to the bug here, we think the correct fix for this bug would be to change what scrolls in the folder pane to only be the actual folders list without the folder pane header. That way scrolling things to be visible will work correctly reliably. This might still need some JS changes, but the big change is having a new scrolling context that doesn't include the header.

Assignee: nobody → martin
Status: NEW → ASSIGNED
Target Milestone: --- → 123 Branch

Pushed by elizabeth@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/62620a3581e5
Fix folder pane scroll root to be the folder list and not the entire sidebar. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Duplicate of this bug: 1863680

good for beta?

Flags: needinfo?(martin)
Flags: needinfo?(martin)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: