major heading error on day view

[Patrick Schaaf]

When using dateDisplay = yyyy-mm-dd (instead of the default), major headings read undefined - 2011-7-9 and also wrap to two lines.

That wrap to two lines, results in the whole right pane (chart area) being shifted down by half a row height, wrt. the left pane (summary, resource, etc).

See the attached picture for details.

[Patrick Schaaf]

Problem happens with both trac 0.11.6 and trac 0.12.2.

On my 0.12.2 test install I also reverted all the changes I made yesterday, the problem remains then same with the plugin out of the current SVN.

Using the default date format I also see an example where 7/10 - 7/16/2011 is displayed (no undefined), and also wraps to two lines, with the same half-row shift between left and right panes as a result.

[Ryan J Ollos]

I'm planning to do some work on improving the date handling this weekend, along with #8988. I can take a look at this ticket if it's still open, assuming it's an issue in Python and not JavaScript.

[Patrick Schaaf]

Replying to rjollos:

I'm planning to do some work on improving the date handling this weekend, along with #8988. I can take a look at this ticket if it's still open, assuming it's an issue in Python and not JavaScript.

It is purely in JavaScript. The python code passes dates regarding the individual rows to the JS, which then calculates all these headings.

[Chris Nelson]

We are almost done with a fix to this. I hope to push the patch this week.

[Chris Nelson]

(In [10480]) Changed major date header to display month year. Refs #8968.

[Ryan J Ollos]

My macro call doesn't specify dateDisplay. When upgrading from r10479 to r10480, the default headings changes.

From:

To:

I'm happy with either format, but thought I'd mention here to confirm that this was the intended change when dateDisplay isn't specified as a macro argument.

[Chris Nelson]

Replying to rjollos:

My macro call doesn't specify dateDisplay. When upgrading from r10479 to r10480, the default headings changes. ... I'm happy with either format, but thought I'd mention here to confirm that this was the intended change when dateDisplay isn't specified as a macro argument.

dateDisplay no longer applies to week labels in day view. This allows a more compact display that won't wrap.

[anonymous]

Upgraded to r10480 without problems, on both 0.11.7 and 0.12.2

I like the new major heading very much - the clear text month name is much more informative!