What version are you running? Whatever's currently in production at VMware. What's the URL of the page containing the problem? Seems to affect any review. What steps will reproduce the problem? 1. Try to edit the description field of a review in Chromium for Linux. What is the expected output? What do you see instead? I expect the text area to only expand when I autowrap to a new line. Instead it does so every time I type a new character. What operating system are you using? What browser? Chromium 18.104.22.168 on Ubuntu 9.04 Please provide any additional information below. It's possible this is a Chromium bug but it doesn't seem to happen on any other webapps.
There are a lot of issues with textareas and the version of WebKit that Chrome is using. It's not just us. We found a forum of people all complaining about similar issues. For 1.0, we are not officially supporting Chrome at all. We recommend Firefox for the best experience. As Chrome matures, we will consider adding it to the support list.
- Priority-Medium + Priority-Low + Browser
I think it's safe to say this is not our bug :)
- Confirmed + ThirdParty
I made this change in djblets, which seems to fix the problem for me. djblets/media/js/jquery.gravy.js: - newHeight = this.element.scrollHeight; + newHeight = this.element.scrollHeight + (this.element.height() - this.element.clientHeight); scrollHeight apparently corresponds to clientHeight, which differs from height() in chrome but not firefox, for me. So...I removed the difference, which I presume is constant and just due to padding.
Hmm, interesting. We'd definitely need to test this at least on Chrome, Safari, IE and Firefox. Have you then tested in both Chrome and Firefox? And it works in both?
- ThirdParty + NeedInfo
Yes, it fixes the issue in both Chrome[ium] and Firefox in Windows and Linux for me. I just grabbed Safari for Windows, and I can't reproduce it in an unpatched install. Not really sure what's going on there. Someone who's not me should definitely try it. :)
Fixed in Djblets on master (r61810c1). Thanks Cory!
- NeedInfo + Fixed
- Priority-Low + Priority-Medium + Djblets + Milestone-Release1.0.x
No, not for a little while. I'll be doing an update pretty soon, but I'm waiting to get a couple more changes in first.