[Rt-devel] [PATCH] css/html layout cleanup

Emmanuel Lacour elacour at easter-eggs.com
Sun Jan 11 12:39:18 EST 2009


Dear RT hackers,


trying to resolve the following tickets, I did a first "rewrite" of RT
css/html. Goal was also to remove absolute/fixed positionning in favour
of dynamic positionning (using floats).

12437: problem with #topactions in konqueror
6823: try to make proper xhtml
12770: #nav in konqueror > 4
10079: display problem when $Showbar = 0


here is some comments of my work:

- tools in search results was not using $actions, but wrote ul/li itself
  with bad positionning, so I moved it to an update of $actions in
  Ticket/Elements/Tab
- 12770: was just a problem with the three "empty" <div> used between
  <ul> and <li> which is not allowed, I just removed them with no side
  effect it seems
- there was some curious ordering in html between page-navigation, nav,
  header and quickbar, I moved them to have a simple layout: a top
  "quickbar", a left "nav", a right "page" (new div, which contains
  topactions, then page-navigation, then body) and a footer
- I don't think RT can be used without scrolling under less than
  1024x768 screens, so I added a min-width of 76em to body to avoid
  "page" to go down "nav" in case of overflow
- I leave the title (mostly ticket subject) in topactions to wrap and so
  use two lines if necessary, this makes the navigation bar to go down if
  needed, but seems far better to me than truncated subjects


I tested this with:

- ie6 win$
- ie7 win$
- ff2 win$
- safari win$
- ff3 GNU/Linux
- konqueror 4 GNU/Linux

12437, 12770 and 10079 seems resolved with this, there doesn't seems to
be major issues (doesn't looks worse than actually ;)).

Can you _test this_ then _report any major regression_ (with screenshots
and navigator version) before I commit this?

I now there is still some little bugs (mostly with ie6 :() and other
thinks I want to enhance, But I'm tired and prefer to commit this as
soon as possible and fix remaining issues one by one :)


PS: attached, test2.html which is the minimal layout I wanted to apply
on RT.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: layout-cleanup.patch
Type: text/x-diff
Size: 16725 bytes
Desc: not available
Url : http://lists.bestpractical.com/pipermail/rt-devel/attachments/20090111/370c463d/attachment.patch 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.bestpractical.com/pipermail/rt-devel/attachments/20090111/370c463d/attachment.html 


More information about the Rt-devel mailing list