Week(Multiday)view: vertical day/date header column too wide, could be much narrower

VERIFIED FIXED

Status

Calendar
Calendar Views
--
trivial
VERIFIED FIXED
12 years ago
11 years ago

People

(Reporter: gekacheka, Assigned: gekacheka)

Tracking

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20061029 Calendar/0.4a1

When week view times are on the horizontal x axis, the day headers are listed vertically in the left column, but arranged using space inefficiently.
 

Reproducible: Always

Steps to Reproduce:
1. View week view
2. transpose so dates and day names are in left column, times across the top.


Actual Results:  
 The date and weekday name are arranged horizontally, so left column is much 
    wider than necessary, wasting a large space below each pair.
  The space for all-day events is too narrow, so only a few letters 
    of the title are visible.

Expected Results:  
 Date and week day names arranged vertically.
 Wider space for all-day events, similar to an hour.
(Assignee)

Comment 1

12 years ago
Created attachment 244061 [details] [diff] [review]
Patch: use vertical boxes for day labels, widen all-day header boxes

(patch -l -p 2 -i file.patch)

Make day label boxes into vboxes, and don't change their orientation.
Widen all-day header boxes.

Patch depends on bug 358688 to move widths to CSS.
Attachment #244061 - Flags: first-review?
(Assignee)

Updated

12 years ago
Depends on: 358688
(Assignee)

Comment 2

12 years ago
Created attachment 244062 [details]
Image showing problem: wide day labels, narrow all-day events boxes
(Assignee)

Comment 3

12 years ago
Created attachment 244063 [details]
Image after patch: narrower day labels, wider all-day events boxes
(Assignee)

Comment 4

12 years ago
Comment on attachment 244061 [details] [diff] [review]
Patch: use vertical boxes for day labels, widen all-day header boxes

mvl, do you agree that when week view has horizontal times and vertical dates, arranging the dates to take less space, and providing more room to display all-day events, improves the UI?
Attachment #244061 - Flags: first-review? → ui-review?(mvl)
Comment on attachment 244061 [details] [diff] [review]
Patch: use vertical boxes for day labels, widen all-day header boxes

ui-review=mvl
Attachment #244061 - Flags: ui-review?(mvl)
Attachment #244061 - Flags: ui-review+
Attachment #244061 - Flags: first-review?(lilmatt)
Created attachment 247865 [details]
Screenshot after applying this patch

gekacheka:
This is what I get after applying this patch.  The date and day are not on top of each other like in your patch, therefore wasting space.  I'm assuming this is wrong, correct?
Comment on attachment 247865 [details]
Screenshot after applying this patch

(In reply to comment #6)
> Created an attachment (id=247865) [edit]

Ignore all of this. I was being a moron. I blame allergies.
Attachment #247865 - Attachment is obsolete: true
Comment on attachment 244061 [details] [diff] [review]
Patch: use vertical boxes for day labels, widen all-day header boxes

r=lilmatt now that I've got my head on straight.
Attachment #244061 - Flags: second-review?(mvl)
Attachment #244061 - Flags: first-review?(lilmatt)
Attachment #244061 - Flags: first-review+
Comment on attachment 244061 [details] [diff] [review]
Patch: use vertical boxes for day labels, widen all-day header boxes

r2=mvl
Attachment #244061 - Flags: second-review?(mvl) → second-review+
Assignee: nobody → gekacheka
Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
VERIFIED with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4pre) Gecko/20070406 Calendar/0.5pre.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.