Closed Bug 692783 Opened 8 years ago Closed 8 years ago

Attendees not aligned with their respective free/busy timelines

Categories

(Calendar :: Dialogs, defect)

Lightning 1.0b7
x86_64
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: psychonaut, Assigned: Paenglab)

Details

Attachments

(3 files, 1 obsolete file)

The "Invite Attendees" dialog contains on the left a list of attendees, and on the right a timeline for each attendee showing their free/busy status.  However, as can be seen in the attached screenshot, the height of the rows in the attendee list is greater than the height of the rows in the timeline, resulting in a misalignment of the two lists.  After more than about a half dozen attendees it becomes difficult or impossible to find their corresponding free/busy information, except by laboriously counting rows starting from the top.

I am using Thunderbird 7.0.1 (installed from the official x86-64 binaries) and Lightning 1.0b7 on Ubuntu 10.04.
Richard, Decathlon, maybe this bug is something for you?
Attached patch Fix the min-height (obsolete) — Splinter Review
I see this issue also with small fonts. Then the freebusy-grid's addressingWidgetItem is 1px less tall than the Attendee list's addressingWidgetItem. This patch adds this missing px.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #565608 - Flags: review?(bv1578)
I can reproduce the issue on Win7 only when I set a small font (e.g. Segoe UI 8pt). With the standard font (9pt) and bigger fonts the misalignment doesn't appear.

The patch fixes the misalignment problem but, with the small font, a withe line appears in the grid (see screenshot).

At first glance a 
 min-height: 16px;
in the .freebusy-grid class fixes the problem, but I didn't look deeply in the issue. Richard, could you please double check?
I think this approach is better and it fixes also the white lines problem with small fonts
Attachment #565608 - Attachment is obsolete: true
Attachment #565608 - Flags: review?(bv1578)
Attachment #568766 - Flags: review?(bv1578)
Comment on attachment 568766 [details] [diff] [review]
Fix the min-height v2

Oh, right. Perfect! :-)

r+
Attachment #568766 - Flags: review?(bv1578) → review+
Pushed to comm-aurora http://hg.mozilla.org/releases/comm-aurora/rev/a1eee289974e

Pushed to comm-beta http://hg.mozilla.org/releases/comm-beta/rev/2b77c4840ab0
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0b8
Btw, what happens with a really large font? Do the grid lines expand?
Yes it expands. Only the min-height was the problem. I tried it with 24px fonts and it worked (and my screen was filled with the list).
You need to log in before you can comment on or make changes to this bug.