General CSS cleanup

ac - October 22, 2009 - 18:35
Project:Hosting
Version:6.x-0.4-alpha2
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

Minor stuff

AttachmentSize
hosting_general_cleanup.patch1003 bytes

#1

ac - October 22, 2009 - 18:37
Status:active» needs review

#2

ac - October 22, 2009 - 19:11

This has actually become a fairly large patch but the file is now very neat. Minor changes needed to eldir which are coming next.

AttachmentSize
remove_css_cruft.patch 4.47 KB

#3

ac - October 22, 2009 - 19:27

Attached a new patch for above and include the eldir patch instead of opening a new issue there.

AttachmentSize
remove_css_cruft(2).patch 4.49 KB
eldir_mods.patch 1.77 KB

#4

adrian - October 22, 2009 - 20:55
Status:needs review» needs work

at the very least the block css still needs to be there in hosting (ie: blocks floated left and right).

it's an important part of providing a usable system out of the box regardless of theme

http://skitch.com/vertice/nrj1k/mytest.dev-t

versus

http://skitch.com/vertice/nrj1c/mytest.dev-t

#5

ac - October 23, 2009 - 03:21

I don't think not floating stuff stops it being usable. I agree its not as pretty but that is why aegir has eldir. Floating stuff makes creating your own theme harder and may cause potential problems with other themes due to the nature of floats. If you wrapped the task list and the platform/site info in a classed div then we could float stuff with more confidence:

<div class="hosting-info-wrapper clear-block">
<div class="hosting-site-info" />
<div class="hosting-task-list" />
</div>

#6

ac - October 23, 2009 - 06:41

modified for latest HEAD

AttachmentSize
hosting_cleanup.patch 4.7 KB

#7

ac - October 23, 2009 - 07:35
Status:needs work» fixed

commited

#8

System Message - November 6, 2009 - 07:40
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.