[prev in list] [next in list] [prev in thread] [next in thread] 

List:       konq-bugs
Subject:    [Bug 79269] Form element with height=1 actually works,
From:       Stephan Kulow <coolo () kde ! org>
Date:       2004-10-30 15:06:48
Message-ID: 20041030150648.23089.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
        
http://bugs.kde.org/show_bug.cgi?id=79269        
coolo kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From coolo kde org  2004-10-30 17:06 -------
CVS commit by coolo: 

ignore height and width on input elements that aren't image submit buttons
BUG: 79269


  M +5 -0      ChangeLog   1.321
  M +6 -6      html/html_formimpl.cpp   1.398


--- kdelibs/khtml/ChangeLog  #1.320:1.321
 @ -1,2 +1,7  @
+2004-10-30  Stephan Kulow  <coolo kde org>
+
+        * html/html_formimpl.cpp (parseAttribute): ignore height element
+        for input elements that are not image
+
 2004-10-28  Stephan Kulow  <coolo kde org>
 

--- kdelibs/khtml/html/html_formimpl.cpp  #1.397:1.398
 @ -1233,9 +1233,9  @ void HTMLInputElementImpl::parseAttribut
         break;
     case ATTR_WIDTH:
-        // ignore this attribute,  do _not_ add
-        // a CSS_PROP_WIDTH here!
-        // webdesigner are stupid - and IE/NS behave the same ( Dirk )
+        if ( m_type == IMAGE )
+            addCSSLength(CSS_PROP_WIDTH, attr->value() );
         break;
     case ATTR_HEIGHT:
+        if ( m_type == IMAGE )
         addCSSLength(CSS_PROP_HEIGHT, attr->value() );
         break;
_______________________________________________
Konq-bugs mailing list
Konq-bugs@mail.kde.org
https://mail.kde.org/mailman/listinfo/konq-bugs
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic