Opened 18 years ago

Closed 13 years ago

Last modified 13 years ago

#4198 closed Bug (fixed)

admin javascript calendar spacing problem

Reported by: Gary Wilson <gary.wilson@…> Owned by: xian
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Flavio Curella Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: yes

Description

See the right edge of the calendar in the attached screen capture.

Setting the table's width to 100% seems to make the gap disappear (at least in Firefox on Linux).

  • django/contrib/admin/media/css/widgets.css

     
    4545.calendarbox, .clockbox { margin:5px auto; font-size:11px; width:16em; text-align:center; background:white; position:relative; }
    4646.clockbox { width:9em; }
    4747.calendar { margin:0; padding: 0; }
    48 .calendar table { margin:0; padding:0; border-collapse:collapse; background:white; width:99%; }
     48.calendar table { margin:0; padding:0; border-collapse:collapse; background:white; width:100%; }
    4949.calendar caption, .calendarbox h2 { margin: 0; font-size:11px; text-align:center; border-top:none; }
    5050.calendar th { font-size:10px; color:#666; padding:2px 3px; text-align:center; background:#e1e1e1 url(../img/admin/nav-bg.gif) 0 50% repeat-x; border-bottom:1px solid #ddd; }
    5151.calendar td { font-size:11px; text-align: center; padding: 0; border-top:1px solid #eee; border-bottom:none; }

So anyone know a reason why this needs to be 99%? Is this needed so that other browsers don't break?

Attachments (1)

4198.diff​ (436 bytes ) - added by Flavio Curella 13 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by Gary Wilson <gary.wilson@…>, 18 years ago

doh, can't attach the screen capture due to ​http://trac.edgewall.org/ticket/4087, so here it is hosted elsewhere:

http://gdub.files.wordpress.com/2007/05/calendar.png

comment:2 by Simon G. <dev@…>, 18 years ago

Triage Stage: Unreviewed → Design decision needed

comment:3 by James Bennett, 17 years ago

Owner: changed from nobody to xian

Reassigning to xian.

comment:4 by Jacob, 17 years ago

Triage Stage: Design decision needed → Accepted

comment:5 by Łukasz Rekucki, 14 years ago

Severity: → Normal
Type: → Bug

comment:6 by Julien Phalip, 13 years ago

UI/UX: set

by Flavio Curella, 13 years ago

Attachment: 4198.diff​ added

comment:7 by Flavio Curella, 13 years ago

Easy pickings: set

Im confirming the incorrect displaying on Safari 5.1 Os X.

I've added a patch, basically just rewriting xian's fix as a diff.

Last edited 13 years ago by Flavio Curella (previous) (diff)

comment:8 by Flavio Curella, 13 years ago

Cc: Flavio Curella added

comment:9 by jamesp, 13 years ago

Triage Stage: Accepted → Ready for checkin

Confirmed fixed on Windows in IE 7, Firefox 5, Chrome, Safari

comment:10 by Julien Phalip, 13 years ago

Resolution: → fixed
Status: new → closed

In [16822]:

Fixed #4198 -- Fixed a small styling issue in the admin calendar widget. Thanks to Gary Wilson and fcurella.

Note: See TracTickets for help on using tickets.
Back to Top