Invalid rows not removed if message list row count drops to 1
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(thunderbird_esr102 unaffected, thunderbird114 fixed)
| Tracking | Status | |
|---|---|---|
| thunderbird_esr102 | --- | unaffected |
| thunderbird114 | --- | fixed |
People
(Reporter: darktrojan, Assigned: darktrojan)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
|
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details | Review |
The logic which removes rows that are no longer valid fails if the only remaining row is row 0, because zero is falsy.
|
Assignee | |
Comment 1•1 year ago
|
||
|
||
Comment 2•1 year ago
|
||
114.0b3
Deleting a message now can cause a scroll of the message list, such that the next message is placed at the bottom of the pane.
Does this patch resolve such an issue?
|
|
Assignee | |
Comment 3•1 year ago
|
||
I very much doubt it. Not sure what that's about, but I've seen it too.
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/e2ef75156d8d
Fix logic error in TreeView._ensureVisibleRowsAreDisplayed. r=leftmostcat
|
|
Assignee | |
Updated•1 year ago
|
|
|
||
Updated•1 year ago
|
|
|
Assignee | |
Updated•1 year ago
|
|
|
||
Comment 5•1 year ago
|
||
Comment on attachment 9334760 [details]
Bug 1833797 - Fix logic error in TreeView._ensureVisibleRowsAreDisplayed. r=leftmostcat
[Triage Comment]
Approved for beta
|
||
Comment 6•1 year ago
|
||
| bugherder uplift | ||
Thunderbird 114.0b5:
https://hg.mozilla.org/releases/comm-beta/rev/98c316bf8d5b
Description
•