<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>18984</bug_id>
          
          <creation_ts>2008-05-10 03:16:40 -0700</creation_ts>
          <short_desc>[GTK] textarea height property works only if other property are defined</short_desc>
          <delta_ts>2009-07-07 08:53:10 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="arno.">a.renevier</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>jmalonzo</cc>
    
    <cc>plaes</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>80059</commentid>
    <comment_count>0</comment_count>
    <who name="arno.">a.renevier</who>
    <bug_when>2008-05-10 03:16:40 -0700</bug_when>
    <thetext>Hi,
When you set height property to a textarea, it is not taken into account.
To make height property work, you must set other properties, such as border, or background-color.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>80060</commentid>
    <comment_count>1</comment_count>
      <attachid>21055</attachid>
    <who name="arno.">a.renevier</who>
    <bug_when>2008-05-10 03:17:50 -0700</bug_when>
    <thetext>Created attachment 21055
testcase</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>80064</commentid>
    <comment_count>2</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2008-05-10 03:58:55 -0700</bug_when>
    <thetext>The same here. But it is only related to WebKitGtk. Changed the report.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>130210</commentid>
    <comment_count>3</comment_count>
      <attachid>32378</attachid>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2009-07-07 08:18:22 -0700</bug_when>
    <thetext>Created attachment 32378
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>130213</commentid>
    <comment_count>4</comment_count>
      <attachid>32378</attachid>
    <who name="Gustavo Noronha (kov)">gustavo</who>
    <bug_when>2009-07-07 08:35:12 -0700</bug_when>
    <thetext>Comment on attachment 32378
patch

Thanks for looking into this! It makes me sad =). I believe leaving this for webcore to do is not really what we want; we probably want to do the same as adjustTextFieldStyle, except for setting the height to auto. I&apos;ll say r- on that basis.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>130218</commentid>
    <comment_count>5</comment_count>
      <attachid>32378</attachid>
    <who name="Gustavo Noronha (kov)">gustavo</who>
    <bug_when>2009-07-07 08:44:16 -0700</bug_when>
    <thetext>Comment on attachment 32378
patch

After clarifying a few concerns with Jan regarding the effect this had in things such as dark themes, I now believe this is correct, so I&apos;ll say r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>130220</commentid>
    <comment_count>6</comment_count>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2009-07-07 08:53:10 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 32378 [details])
&gt; After clarifying a few concerns with Jan regarding the effect this had in
&gt; things such as dark themes, I now believe this is correct, so I&apos;ll say r=me.

Thanks. Landed in r45595.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>21055</attachid>
            <date>2008-05-10 03:17:50 -0700</date>
            <delta_ts>2008-05-10 03:17:50 -0700</delta_ts>
            <desc>testcase</desc>
            <filename>18984.html</filename>
            <type>text/html</type>
            <size>309</size>
            <attacher name="arno.">a.renevier</attacher>
            
              <data encoding="base64">PGJvZHk+Cjx0ZXh0YXJlYSBpZD0ib25lIiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjogcmVkOyBo
ZWlnaHQ6IDEwMHB4Ij5iYWNrZ3JvdW5kLWNvbG9yOiByZWQ7IGhlaWdodDogMTAwcHg8L3RleHRh
cmVhPgo8dGV4dGFyZWEgaWQ9Im9uZSIgc3R5bGU9ImJvcmRlcjogMXB4IHNvbGlkIGdyZWVuOyBo
ZWlnaHQ6IDEwMHB4Ij5ib3JkZXI6IDFweCBzb2xpZCBncmVlbjsgaGVpZ2h0OiAxMDBweDwvdGV4
dGFyZWE+Cjx0ZXh0YXJlYSBpZD0idHdvIiBzdHlsZT0iaGVpZ2h0OiAxMDBweCI+aGVpZ2h0OiAx
MDBweDwvdGV4dGFyZWE+CjwvYm9keT4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32378</attachid>
            <date>2009-07-07 08:18:22 -0700</date>
            <delta_ts>2009-07-07 08:44:16 -0700</delta_ts>
            <desc>patch</desc>
            <filename>textareafix.patch</filename>
            <type>text/plain</type>
            <size>3266</size>
            <attacher name="Jan Alonzo">jmalonzo</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA4YmE0NDJkLi4yMDJmNjE3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMDktMDct
MDcgIEphbiBNaWNoYWVsIEFsb256byAgPGptYWxvbnpvQHdlYmtpdC5vcmc+CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgR1RLXSB0ZXh0YXJlYSBoZWln
aHQgcHJvcGVydHkgd29ya3Mgb25seSBpZiBvdGhlciBwcm9wZXJ0eSBhcmUgZGVmaW5lZAorICAg
ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTg5ODQKKworICAg
ICAgICBMZXQgV2ViQ29yZSBoYW5kbGUgdGV4dGFyZWEncyBtZXRyaWNzIGluc3RlYWQgb2YgcmVh
ZGp1c3RpbmcgaXQgUmVuZGVydGhlbWVHdGsuCisKKyAgICAgICAgVGVzdDogZmFzdC9mb3Jtcy90
ZXh0YXJlYS1tZXRyaWNzLmh0bWwKKworICAgICAgICAqIHBsYXRmb3JtL2d0ay9Ta2lwcGVkOgor
CiAyMDA5LTA3LTA3ICBCZW4gTXVyZG9jaCAgPGJlbm1AZ29vZ2xlLmNvbT4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBBbnR0aSBLb2l2aXN0by4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsYXRm
b3JtL2d0ay9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1NraXBwZWQKaW5kZXgg
MjM5YjhmMy4uZjIxNjQ3MiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Nr
aXBwZWQKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1NraXBwZWQKQEAgLTE3ODYsNyAr
MTc4Niw2IEBAIGZhc3QvZm9ybXMvc2xpZGVyLXRyYW5zZm9ybWVkLmh0bWwKIGZhc3QvZm9ybXMv
c2xpZGVyLXpvb21lZC5odG1sCiBmYXN0L2Zvcm1zL3RleHQtY29udHJvbC1pbnRyaW5zaWMtd2lk
dGhzLmh0bWwKIGZhc3QvZm9ybXMvdGV4dGFyZWEtaW5wdXQtZXZlbnQuaHRtbAotZmFzdC9mb3Jt
cy90ZXh0YXJlYS1tZXRyaWNzLmh0bWwKIGZhc3QvZm9ybXMvdGV4dGFyZWEtcGFzdGUtbmV3bGlu
ZS5odG1sCiBmYXN0L2Zvcm1zL3RleHRhcmVhLXNjcm9sbGJhci1oZWlnaHQuaHRtbAogZmFzdC9m
b3Jtcy90ZXh0YXJlYS1zY3JvbGxlZC1lbmRsaW5lLWNhcmV0Lmh0bWwKZGlmZiAtLWdpdCBhL1dl
YkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOGQ5NzE1Yi4uNzgyMWJm
OSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cK
QEAgLTEsMyArMSwxNyBAQAorMjAwOS0wNy0wNyAgSmFuIE1pY2hhZWwgQWxvbnpvICA8am1hbG9u
em9Ad2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBHVEtdIHRleHRhcmVhIGhlaWdodCBwcm9wZXJ0eSB3b3JrcyBvbmx5IGlmIG90aGVy
IHByb3BlcnR5IGFyZSBkZWZpbmVkCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD0xODk4NAorCisgICAgICAgIExldCBXZWJDb3JlIGhhbmRsZSB0ZXh0YXJl
YSdzIG1ldHJpY3MgaW5zdGVhZCBvZiByZWFkanVzdGluZyBpdCBSZW5kZXJ0aGVtZUd0ay4KKwor
ICAgICAgICBUZXN0OiBmYXN0L2Zvcm1zL3RleHRhcmVhLW1ldHJpY3MuaHRtbAorCisgICAgICAg
ICogcGxhdGZvcm0vZ3RrL1JlbmRlclRoZW1lR3RrLmNwcDoKKyAgICAgICAgKiBwbGF0Zm9ybS9n
dGsvUmVuZGVyVGhlbWVHdGsuaDoKKwogMjAwOS0wNy0wNyAgQmVuIE11cmRvY2ggIDxiZW5tQGdv
b2dsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQW50dGkgS29pdmlzdG8uCmRpZmYgLS1n
aXQgYS9XZWJDb3JlL3BsYXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5jcHAgYi9XZWJDb3JlL3Bs
YXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5jcHAKaW5kZXggMDk4NDk3Yy4uZmRlZjljMiAxMDA2
NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9ndGsvUmVuZGVyVGhlbWVHdGsuY3BwCisrKyBiL1dl
YkNvcmUvcGxhdGZvcm0vZ3RrL1JlbmRlclRoZW1lR3RrLmNwcApAQCAtMzIwLDExICszMjAsNiBA
QCBib29sIFJlbmRlclRoZW1lR3RrOjpwYWludFRleHRGaWVsZChSZW5kZXJPYmplY3QqIG8sIGNv
bnN0IFJlbmRlck9iamVjdDo6UGFpbnRJbgogICAgIHJldHVybiBwYWludE1veldpZGdldCh0aGlz
LCBNT1pfR1RLX0VOVFJZLCBvLCBpLCByZWN0KTsKIH0KIAotdm9pZCBSZW5kZXJUaGVtZUd0azo6
YWRqdXN0VGV4dEFyZWFTdHlsZShDU1NTdHlsZVNlbGVjdG9yKiBzZWxlY3RvciwgUmVuZGVyU3R5
bGUqIHN0eWxlLCBFbGVtZW50KiBlKSBjb25zdAotewotICAgIGFkanVzdFRleHRGaWVsZFN0eWxl
KHNlbGVjdG9yLCBzdHlsZSwgZSk7Ci19Ci0KIGJvb2wgUmVuZGVyVGhlbWVHdGs6OnBhaW50VGV4
dEFyZWEoUmVuZGVyT2JqZWN0KiBvLCBjb25zdCBSZW5kZXJPYmplY3Q6OlBhaW50SW5mbyYgaSwg
Y29uc3QgSW50UmVjdCYgcikKIHsKICAgICByZXR1cm4gcGFpbnRUZXh0RmllbGQobywgaSwgcik7
CmRpZmYgLS1naXQgYS9XZWJDb3JlL3BsYXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5oIGIvV2Vi
Q29yZS9wbGF0Zm9ybS9ndGsvUmVuZGVyVGhlbWVHdGsuaAppbmRleCA2NmY3YWUyLi4xM2RhYWEy
IDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsYXRmb3JtL2d0ay9SZW5kZXJUaGVtZUd0ay5oCisrKyBi
L1dlYkNvcmUvcGxhdGZvcm0vZ3RrL1JlbmRlclRoZW1lR3RrLmgKQEAgLTg5LDcgKzg5LDYgQEAg
cHJvdGVjdGVkOgogICAgIHZpcnR1YWwgdm9pZCBhZGp1c3RUZXh0RmllbGRTdHlsZShDU1NTdHls
ZVNlbGVjdG9yKiwgUmVuZGVyU3R5bGUqLCBFbGVtZW50KikgY29uc3Q7CiAgICAgdmlydHVhbCBi
b29sIHBhaW50VGV4dEZpZWxkKFJlbmRlck9iamVjdCosIGNvbnN0IFJlbmRlck9iamVjdDo6UGFp
bnRJbmZvJiwgY29uc3QgSW50UmVjdCYpOwogCi0gICAgdmlydHVhbCB2b2lkIGFkanVzdFRleHRB
cmVhU3R5bGUoQ1NTU3R5bGVTZWxlY3RvciosIFJlbmRlclN0eWxlKiwgRWxlbWVudCopIGNvbnN0
OwogICAgIHZpcnR1YWwgYm9vbCBwYWludFRleHRBcmVhKFJlbmRlck9iamVjdCosIGNvbnN0IFJl
bmRlck9iamVjdDo6UGFpbnRJbmZvJiwgY29uc3QgSW50UmVjdCYpOwogCiAgICAgdmlydHVhbCB2
b2lkIGFkanVzdE1lbnVMaXN0U3R5bGUoQ1NTU3R5bGVTZWxlY3RvciosIFJlbmRlclN0eWxlKiwg
RWxlbWVudCopIGNvbnN0Owo=
</data>
<flag name="review"
          id="16832"
          type_id="1"
          status="+"
          setter="gustavo"
    />
          </attachment>
      

    </bug>

</bugzilla>