Bug 221830

Created attachment 39696 [details]
patch

Thanks for the report and patch. The issue has been reported a while ago already.

*** This bug has been marked as a duplicate of bug 173585 ***

@andrea: fwiw, I agree with the patch. The idea initially was to have form widgets clearly identified in any circumstances, has an accessibility helper.

This is no longer much of an issue since we have removed most of the other styling limitations on other widgets, so this is now inconsistent (until I get around introducing that accessibility option I think about)

could you possibly get an svn account and commit this yourself?
thanks

> could you possibly get an svn account and commit this yourself?
> thanks

Thank you for the trust, but probably will not right away....
In the meantime, you can apply the patch if you want.

Cheers,
Andrea.

SVN commit 1090000 by aiacovitti:

Do not display scroll bars on textarea elements when not necessary.
Match other browsers behaviour.
Approved by Germain.

BUG: 221830
CCBUG: 173585


 M  +7 -8      render_form.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1090000

thanks for applying eventually and commiting the fix, sorry I didn't respond beforehand.
As you might guess this isn't so much a matter of trust (KDE doesn't have much of a commit-access-is-sacred mindset ; it rather put the emphasis on responsibility for one's commits) as a matter of being really very few people to handle patches.
If you haven't noticed, KHTML hacking is unpopular ; it annoys a lot of people.
*Don Giovanni's Commantore theme*

SVN commit 1093037 by ggarand:

automatically merged revision 1090000:
Do not display scroll bars on textarea elements when not necessary.
Match other browsers behaviour.
Approved by Germain.

BUG: 221830
CCBUG: 173585

 M  +7 -8      render_form.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1093037

SVN commit 1097847 by ggarand:

got a better idea here:

instead of letting the textarea be always ScrollBarAsNeeded,
we may mirror the overflow-x/y property and default to overflow-y:scroll

This matches MSIE7 so should be perfectly compatible with sites that want
to turn the scrollbars to auto for cosmetic reasons (with overflow:auto),
while retaining as default the very nice "this is a form widget"
accessibility hint that the vertical scrollbar naturally provides.

CCBUG: 221830

 M  +1 -0      css/html4.css  
 M  +13 -0     rendering/render_form.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1097847