351902 - 15 and 30 minute events barely usable on many displays

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Dan Mosedale (:dmosedale, :dmose)]

Since the 24 hour views have landed, it's really difficult to see and use 15/30 minute events.  In the slightly longer term, there are some design issues to be dealt with.  For 0.3, however, we propose to try a workaround such that the number of pixels per minute is set differently in the day and week views, in the theory that people sometimes want the overview (weekview), and sometimes the zoomed view where everything is visible.

Comment 1

[Matthew (lilmatt) Willis]

So what do you want to change for each view?
Do you want to increase the pixPerMin on day view?

I'm not clear on which view needs to grow/shrink...

http://lxr.mozilla.org/mozilla/source/calendar/base/content/calendar-multiday-view.xml#63

Comment 2

[Thomas Benisch]

The relevant fields are

      <field name="mMinHorizontalPixelsPerMinute">1.2</field>
      <field name="mMinVerticalPixelsPerMinute">0.5</field>

, see also

http://lxr.mozilla.org/mozilla/source/calendar/base/content/calendar-multiday-view.xml#1651

At the moment those values are the same for the day and week
view. If I understand dmose correct, the idea is to increase
the values for the day view.

Comment 3

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

In order to make a 15 minute item readable, mMinVerticalPixelsPerMinute needs to be around 1.5. But at this size, you can only see about 4 hours (depending on screensize etc, ofcourse) I think a change that big is not what we want.
A value of 0.7 at least makes 30 minute events have a readable title. That might be a better way to go.

Comment 4

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

Attachment: change value

Patch makes the value for dayview slightly higher.

Comment 5

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 238914 [details] [diff] [review]
change value

Looks good; two-levels of review on a bug this trivial isn't likely to benefit the patch or the reviewers. r1/r2=dmose.

Comment 6

[Joey Minta]

Comment on attachment 238914 [details] [diff] [review]
change value

                 var viewElement = document.getAnonymousElementByAttribute(
                                   this, "anonid", "view-element");
                 viewElement.daysOffArray = [];
 
                 // add a preference observer to monitor changes
                 var pb2 = Components.classes
                           ["@mozilla.org/preferences-service;1"]
                           .getService(Components.interfaces.nsIPrefBranch2);
                 pb2.addObserver("calendar.", this.mPrefObserver, false);
+
+                var viewElement = document.getAnonymousElementByAttribute(this, "anonid", "view-element");
+
This results in a double-declaration of viewElement.

Comment 7

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

Patch (with jminta's comment fixed) checked in