Task list in Today Pane doesn't remember sort column and sort order
Categories
(Calendar :: General, defect)
Tracking
(Not tracked)
People
(Reporter: ssitter, Assigned: pmorris)
References
Details
(Keywords: regression)
Attachments
(2 files)
|
(deleted),
patch
|
darktrojan
:
review+
darktrojan
:
approval-calendar-beta+
darktrojan
:
approval-calendar-esr+
|
Details | Diff | Splinter Review |
|
(deleted),
image/png
|
Details |
sorted-not-sorted.png
Tested with Thunderbird 68.1.0 (Build ID 20190909201201) and Thunderbird 71.0a1 (Build ID 20190914224146).
STR: Enable task list in today pane and sort tasks by e.g. Due Date column, ascending order. Restart Thunderbird and check task list.
Actual result: Task list in today pane is not sorted.
Expected result: Task list in today pane is sorted as configured before closing Thunderbird.
|
Reporter | |
Comment 1•5 years ago
|
||
Paul, in Bug 1545199 you have been working on restoring column visibility and column order. Maybe you have an idea why its not working for sort column / sort order?
|
Assignee | |
Comment 2•5 years ago
|
||
Thanks for the report. Here's the simple fix.
|
||
Comment 3•5 years ago
|
||
Comment on attachment 9093438 [details] [diff] [review] persist-task-sort-order-0.patch This looks like the sort of mistake I'd make! r+ assuming you've tested this is the actual cause.
|
|
Assignee | |
Comment 4•5 years ago
|
||
Comment on attachment 9093438 [details] [diff] [review] persist-task-sort-order-0.patch Yep, tested to confirm that this was the cause and the fix. And yeah, we're not really doing ourselves any favors by using both camel case and kebab case names (sortActive and sort-active). Forgot to request approvals.
|
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 6•5 years ago
|
||
Yep, thanks for the reminder.
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/76f68ab2dceb
Persist task list sort column and sort order. r=darktrojan
|
||
Updated•5 years ago
|
|
|
||
Comment 8•5 years ago
|
||
TB 70 beta 2:
https://hg.mozilla.org/releases/comm-beta/rev/968907b79abda93ef567ef6a178a790ae2bd5cba
|
|
||
Comment 9•5 years ago
|
||
TB 68.1.1 ESR / Cal 7.0:
https://hg.mozilla.org/releases/comm-esr68/rev/1b244cd88a7fbcc8f9aa5ff1644fa4286ab7aa0d
|
|
Reporter | |
Comment 10•5 years ago
|
||
Reopen because it does not work using Lightning 68.1.1 (Build ID 20190924105435, candidates/build1).
After restart of Thunderbird the column is marked with the sort arrow but the actually list content is not sorted, see screenshot.
|
|
Reporter | |
Updated•5 years ago
|
|
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 11•5 years ago
|
||
Given that this is working on trunk, I was able to fix it on 68.1.1 by porting the patches from bug 1568723 and then bug 1569513 to 68 (fixing prettier formatting conflicts). But with those two patches applied, the todaypane is broken in several ways. So there must be another patch to port that these patches depend on. That's the next step. On the bright side, porting these patches should improve performance on 68.
|
|
Assignee | |
Comment 12•5 years ago
|
||
Spoke too soon. Turns out it was just a one-line change that got lost during the port of bug 1568723. After fixing that the todaypane works as expected. I'll upload those ported patches to their respective bugs, and flag darktrojan for review and uplift approval.
(For some reason my 68 build is not finishing the job with Lightning. I have to manually rename the xpi file and then copy it over into the profile directory. This tripped me up because until I realized I needed to manually copy the file into the profile directory, Lightning didn't have the code changes.)
|
|
||
Comment 13•5 years ago
|
||
Excellent! Let's call this fixed since it's working on trunk and will be fixed elsewhere through the appropriate backports.
|
|
Reporter | |
Updated•5 years ago
|
Description
•