Closed Bug 1598885 Opened 5 years ago Closed 5 years ago

Lightning no longer shows tasks due today in bold

Categories

(Calendar :: Tasks, defect)

Lightning 68
x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
68.3.1

People

(Reporter: mozilla, Assigned: Paenglab)

Details

(Keywords: regression)

Attachments

(2 files)

Up until now, lightning used to show tasks due today in a bold font in the list. This made it very easy to spot what is on the spot today.
However, now today's tasks are shown in the same font as the tasks further due in the future. Only tasks due in the past are shown differently, using a red font. Is there any way to get the old behavior back

Attached image task-list-style.png โ€”

Confirmed.

Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression

Richard, do you happen to know if this style change was done on purpose? If not, would it be suitable to restore this styling?

Screenshot shows a 2nd regression regarding styling: In both images no task is selected and focus is not within today pane. But in Tb 68 there is this blue focus rectangle that will always be displayed.

Flags: needinfo?(richard.marti)

It's not done on purpose AFAIK. maybe this is a regression from the de-XBL work. Because it's a tree, I can't inspect it to check what is set.

I don't see the 2nd regression locally. Again, trees aren't inspectable and the rectangle is normally shown when the treechildren has the attribute [current].

Flags: needinfo?(richard.marti)
Attached patch 1598885-task-duetoday.patch โ€” โ€” Splinter Review

Looked in the file history and found that I was the sinner in bug 1504190. :-(

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9112044 - Flags: review?(paul)
Attachment #9112044 - Flags: approval-calendar-esr?(paul)
Attachment #9112044 - Flags: approval-calendar-beta?(paul)
Comment on attachment 9112044 [details] [diff] [review] 1598885-task-duetoday.patch Review of attachment 9112044 [details] [diff] [review]: ----------------------------------------------------------------- Changes look good, and I confirmed that tasks that are due today appear with bold text.
Attachment #9112044 - Flags: review?(paul)
Attachment #9112044 - Flags: review+
Attachment #9112044 - Flags: approval-calendar-esr?(paul)
Attachment #9112044 - Flags: approval-calendar-esr+
Attachment #9112044 - Flags: approval-calendar-beta?(paul)
Attachment #9112044 - Flags: approval-calendar-beta+

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/b56719530ba2
Re-add the rule for the duetoday tasks. r=pmorris

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

73

Flags: needinfo?(philipp)
Target Milestone: --- → 72
Flags: needinfo?(philipp)
Target Milestone: 72 → 73
Target Milestone: 72 → 68.3.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: