781424 - events in today pane could be one-liners

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, enhancement)

Description

[hm]

User Agent: Mozilla/5.0 (X11; FreeBSD i386; rv:14.0) Gecko/20120805 Firefox/14.0.1 SeaMonkey/2.11
Build ID: 20120805074345

Steps to reproduce:

by default my events in today pane show up as two-liners under date line



Expected results:

you could consider for better readability and having more events on screen, changing this behavior into showing events as one-liner, similar to task list

Comment 1

[Markus Adrario [:Taraman]]

That is a valid proposal.

Since I don't know why it was done like it is now, setting uiwanted to get it discussed.

Comment 2

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Probably to make it easier in the "Soon" section, because here you will need the full date, which will take up the whole line.

Ideally, we could make the boxes wrap automatically so that if the time/title is short enough it would fit on one line, otherwise it would wrap into two.

I don't know how easy that is to manage with XUL though.

Comment 3

[Markus Adrario [:Taraman]]

The Today pane utilizes the agenda-richlist for the display.
At the moment, it is the only place where the agenda is used.

Are there any thoughts of adding an "agenda-view" in the future? Then we should keep in Mind that the same code would also apply to this view.

I'll have a look, if the auto-wrap can be done though.

Comment 4

[Markus Adrario [:Taraman]]

Attachment: Proposal for more compact Today-pane agenda

Since Philipp guessed it right and automatic wrap with XUL would mean a lot of workaround - if really achievable, I tried a different approach.

for items on "today" and "tomorrow", only the duration box is used and the titlebox is hidden.
For the soon-section with long dates, we get a 2-line display as is now.

Why?:
- Long titles are also cut off with current Lightnig. This solution will only crop 7 more characters. 
- normally you will have a good idea whats happening the next days, so you will know the event even with the title cut off.
- If we get an agenda-view sometime, it will be wider and this design will go well with that.

Comment 5

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: wrap text - v1

I just hacked this together shortly because Ms2ger asked about it. This breaks the allday event boxes and causes the allday area to scroll, but it would at least allow wrapping the text somehow. Maybe some special casing could be done for the allday items or the other way around for agenda items by using css -moz-box-flex instead of the flex attribute.

Maybe you can continue on this patch? If it doesn't work out we can still go with the other solution.

Comment 6

[Markus Adrario [:Taraman]]

Comment on attachment 766923 [details] [diff] [review]
wrap text - v1

This looks good. The textContent Attribute is not documented in MDN xul-label page...

This patch also changes the Multiday-View boxes to wrap - was this intended?
On my lightning where I keep the appointments of all my family this would greatly reduce the readability.

I will continue with this but only for the agenda-view, as this is the scope of the bug.

Comment 7

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

textContent is the general DOM attribute for content of elements/nodes, probably thats why its not documented there. We should add a note though.

Anyway, this was hacked together, the side effects are not necessarily intended. If there is a way to make it wrap for multiday view I don't think it would be so bad though. The problem with multiday view though is that the box size are very delicate. The box must not change size because the text wraps.

Comment 8

[Stefan Sitter [:ssitter]]

Don't make me scrolling in the calendar views just because the title of an event is too long.

If reusing the existing code from calendar view is showing to be complicated, why not create simple new code that is targeted for the agenda list view and its requirements?

Comment 9

[Markus Adrario [:Taraman]]

Attachment: proposed look of today-pane agenda

I also don't want the multiday view get multiline events.
So I keep it to the Today-Pane.

This is how it looks so far.
It still needs some testing and we need to decide if we wrap the allday-events too.

Comment 10

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

(In reply to Stefan Sitter from comment #8)
> Don't make me scrolling in the calendar views just because the title of an
> event is too long.
Yes, this is a bug from the wip patch. Not intended.

Comment 11

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

(In reply to Markus Adrario [:Taraman] from comment #9)

> This is how it looks so far.
> It still needs some testing and we need to decide if we wrap the
> allday-events too.
lgtm. I would wrap allday events in the today pane. If you can make changes so it also works in the multiday view without making the scrollbar appear, then we might save the work of creating a separate binding with some duplicate code.

Comment 12

[Markus Adrario [:Taraman]]

Attachment: new look with allday-events wrapping

I managed to only get the allday-items wrap in today-pane.

This is what it will look.
Patch to follow shortly

Comment 13

[Markus Adrario [:Taraman]]

Attachment: Textwrap-Patch V1

Comment 14

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 769114 [details] [diff] [review]
Textwrap-Patch V1

Review of attachment 769114 [details] [diff] [review]:
-----------------------------------------------------------------

Pretty cool, r=philipp

Comment 15

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Textwrap Patch - v2

Here is the patch for checkin, which is the same as v1 but with correct patch header and commit message set.

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/331134c6846e