[drupal-devel] [bug] No more floating node admin groups

moshe weitzman drupal-devel at drupal.org
Tue Aug 2 19:35:51 UTC 2005


Issue status update for 
http://drupal.org/node/26311
Post a follow up: 
http://drupal.org/project/comments/add/26311

 Project:      Drupal
 Version:      4.6.0
 Component:    base system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Bèr Kessels
 Updated by:   moshe weitzman
 Status:       patch (code needs review)

I like the compactness of the current approach ... If you feel that
themers will not know how to override this, then please add some
comments to drupal.css or elswhere.




moshe weitzman



Previous comments:
------------------------------------------------------------------------

Mon, 04 Jul 2005 08:56:49 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/non_floating_node_admin.patch (835 bytes)

For me -konqueror user- the floats on the "admin groups" of the node
edit  forms was always annoying. The textfields were inaccessible.
But witth the collapsible parts they becme even moer annyoing: They now
jump around on teh page when collapsed or folded out. 


This patch simply removes all the floats frolm these groups. 


I beleive theis looks and feels much better. It appears more stable on
your page. It feels more like one expects: only a single row of
collapses, instead of some odd row with columns. 


And last but not least: that part was amoungst the hardests to theme. I
think this is a good moment to get rid of them. 


Oh, and it removes quite some code/cruft from drupal.css :)




------------------------------------------------------------------------

Mon, 04 Jul 2005 09:04:14 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/collapsible_floats3.png (178.97 KB)

here is an example of the jumping fieldsets: A complete set of
screenshots can be found on my computer:
http://webschuur.dyndns.org/DMP/collapsible/




------------------------------------------------------------------------

Mon, 04 Jul 2005 09:06:20 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/collapsible_floats_new_7.png (13.07 KB)

and here is how it looks with the patch.




------------------------------------------------------------------------

Mon, 04 Jul 2005 11:59:32 +0000 : Thox

This seems a fair temporary fix for Konqueror, although it will affect
all browsers. In the long term we need to improve the layout of the
form itself.




------------------------------------------------------------------------

Mon, 04 Jul 2005 13:22:49 +0000 : Bèr Kessels

AllthoughI presented it as a fix for myself and konqueror. I meant it as
a fix in general. IMO the floated collapsed forms look and feel strange,
on konqueror particularly, bu in firefox too. Having them underneath
eachother is much nicer and friendlier. Worpress has that too: one
collapse per line.




------------------------------------------------------------------------

Tue, 02 Aug 2005 18:55:37 +0000 : Bèr Kessels

This pathc still applies.
IMO we *really* do not ned floating elements with the current new
collapsing form elements. So can someone please comment on this patch?
If you think you really need the floating form elements, please tell
why. And if you dont think you need it, please try and comment on this
patch.







More information about the drupal-devel mailing list