[drupal-devel] [bug] illegible form elements for content administration (Safari)

[adrinux]

Issue status update for http://drupal.org/node/17343

 Project:      Drupal
 Version:      cvs
 Component:    node.module
 Category:     bug reports
 Priority:     critical
 Assigned to:  Anonymous
 Reported by:  njivy
 Updated by:   adrinux
 Status:       patch

I can't see any other way to fix that, adding an ID and setting height
on the fieldset would be just as ugly imo - maybe re-engineering the
whole form would work but this isn't really the time to do that! Steven
will moan at you for using a br probably :)


adrinux



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

February 14, 2005 - 23:43 : njivy

Attachment: http://drupal.org/files/issues/content_administration-Safari.jpg (33.21 KB)

Using Safari 1.2.4 under OS 10.3, the form elements for content
administration are not legible.  (The path is "admin/node".)  I am not
enough of a CSS whiz to directly propose a solution.  But I am willing
to test patches.
I attached a screenshot.


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

February 15, 2005 - 14:43 : njivy

Oops. Wrong version.


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

February 25, 2005 - 01:31 : mathias

This has been fixed [1].
[1] http://drupal.org/cvs?commit=13633


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

March 27, 2005 - 05:00 : adrian

Attachment: http://drupal.org/files/issues/safari_error.jpg (11.03 KB)

I just found the admin menu similarly messed up in the 4.6RC. 
I flushed my drupal.css just in case, but the bug definitely persist.


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

March 27, 2005 - 08:10 : Junyor

The same thing is happening in Opera.  I tracked it down to having a
floated object inside a fieldset.  Firefox uses the fieldset as the
containing block for the float, but Opera and Safari move the floated
content outside of the fieldset.


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

March 27, 2005 - 12:09 : Goba

So then a container div might be a working solution.


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

March 27, 2005 - 12:58 : adrian

Attachment: http://drupal.org/files/issues/safari_float_render.diff (1.29 KB)

Here is a patch that adds a break , with a class of 'clear' , and adds a
.clear class to the drupal.css file.
This is how I would fix it, but it's probably not semantically correct.
But it does fix the problem..


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

March 27, 2005 - 12:58 : adrian

err. and it's a patch.


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

March 27, 2005 - 13:37 : adrian

Attachment: http://drupal.org/files/issues/safari_float_render_0.diff (1.29 KB)

I'm using the coffee defense as to why I missed the . infront of the
.clear class