Closed Bug 1862197 Opened 1 year ago Closed 9 months ago

Expanding thread/group header in message view scrolls too much

Categories

(Thunderbird :: Folder and Message Lists, defect)

Thunderbird 120
defect

Tracking

(thunderbird_esr115+ fixed, thunderbird120 wontfix, thunderbird124 wontfix)

RESOLVED FIXED
125 Branch
Tracking Status
thunderbird_esr115 + fixed
thunderbird120 --- wontfix
thunderbird124 --- wontfix

People

(Reporter: bugzilla.mozilla.org, Assigned: welpy-cw)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Steps to reproduce:

STR:

  • Enable "Grouped by Sort" in message list view (or message card view)
  • Shrink and then expand a header (like the "Today" header)

Actual results:

The "Today" header is scrolled out of view

Expected results:

The header that is expanded should be scrolled to the top of the message list view - but not out of view.

Summary: Expanding group header in message view incorrectly scrolls too much → Expanding group header in message view scrolls too much
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
Regressed by: tb-new-3pane
Summary: Expanding group header in message view scrolls too much → Expanding thread/group header in message view scrolls too much
Assignee: nobody → h.w.forms
Status: NEW → ASSIGNED
Severity: -- → S3
Target Milestone: --- → 125 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/601bf491aa5c
Correct scroll position when expanding a row. r=darktrojan

Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED

Comment on attachment 9361228 [details]
Bug 1862197 - Correct scroll position when expanding a row. r=#thunderbird-front-end-reviewers

[Approval Request Comment]
Testing completed (on c-c, etc.): yes
Risk to taking this patch (and alternatives if risky): very low

Attachment #9361228 - Flags: approval-comm-esr115?
Component: Mail Window Front End → Folder and Message Lists

Comment on attachment 9361228 [details]
Bug 1862197 - Correct scroll position when expanding a row. r=#thunderbird-front-end-reviewers

[Triage Comment]
Approved for esr115

Has tests and no regressions reported. Thanks!

Attachment #9361228 - Flags: approval-comm-esr115? → approval-comm-esr115+

The original patch is causing test failures. I recommend backing out that revision and checking in this one instead.

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

Attachment

General

Created:
Updated:
Size: