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

[Bèr Kessels]

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:   Bèr Kessels
-Status:       patch (code needs review)
+Status:       patch (ready to be committed)
 Attachment:   http://drupal.org/files/issues/uglyfloats.png (53.75 KB)

the curertn floats have not other reason to be there, then leftovers
from teh pre-collapsible-era.
If we cannot do bettert then it is now, then at least removing the
floats makes it all behave better. 


See the att. screenshot: the comment options are "lost", they are hard
to be found by a pesron scanning the UI.. The top-element does not fill
out the full width (just looks ugly and inconsistent) and
collapsing/uncollapsing makes this all behave even stranger.




Bèr Kessels



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.




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

Tue, 02 Aug 2005 19:35:44 +0000 : moshe weitzman

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.