325399 - tweak CSS padding to cut down on font-clipping in event boxes

Status: RESOLVED FIXED

(Calendar :: Internal Components, defect)

Description

[Dan Mosedale (:dmosedale, :dmose)]

This should be an extremely way to improve visual effect before releasing Lightning 0.1.  Spun off from bug 322981.

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

s/extremely/extremely easy

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: first try (checked in)

Not sure if this is what you have in mind, but it does allow for more events-per-day in the month and multiweek views.
(we really need to fix our theme story...)

Comment 3

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 210527 [details] [diff] [review]
first try (checked in)

That's pretty much exactly what I had in mind; thanks!  Can I talk you into doing the multiday view also?

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 210527 [details] [diff] [review]
first try (checked in)

patch checked in.
I'm not sure what to with the multiday view, becuase it has much more room. No need to remove the margins there, i think. But leaving the bug open in case you disagree.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: updates (checked in)

The checkin for bug 321546 made the event label have a bigger font again, while the time was still small. The labels also no longer aligned properly
This patch corrects that.

Comment 6

[Joey Minta]

Comment on attachment 210703 [details] [diff] [review]
updates (checked in)

When we edit, we use font-size: x-small, http://lxr.mozilla.org/mozilla/source/calendar/base/content/calendar-multiday-view.css#92 instead of the 0.9em you use here.  I'm not sure if it's better to match those up or not. r=jminta either way.

Comment 7

[Dan Mosedale (:dmosedale, :dmose)]

mvl has convinced me that trying to do the same trick for the week views would make them look strange for the default settings.  I see lots of extra clipping because I display 0700 through 2300.

Comment 8

[Reed Loden [:reed]]

The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!