Tommy Kuo (away forever...) Reporter
	Description • 8 years ago

I think it would be a better experience when the height of the textarea of "add comment" would be resized automatically. I think it is a bad experience when we type a long comment and we have to scroll the little textarea to check it.

And the height of "preview" should be "auto", too. It makes the comment editor can preview his/her entire comment quickly.

	Tommy Kuo (away forever...) Reporter
	Comment 1 • 8 years ago

I created a patch to make the comment textarea be resized automatically.

	William Lachance (:wlach)
	Comment 2 • 7 years ago

I'm going to +1 this, I frequently find the textarea to be too small when I'm editing anything but a very short comment. I think we might want to set some kind of upper limit on its size (I also commented to this effect on the PR)

	Dylan Hardison [:dylan] (he/him)
	Comment 3 • 7 years ago

Comment on attachment 8827765 [details] [review]
github pull request

Add a max height and I think this is a useful change.

	Tommy Kuo (away forever...) Reporter
	Comment 4 • 7 years ago

I updated the patch with setting 80% height of the viewport as the max height.

	Dylan Hardison [:dylan] (he/him)
	Comment 5 • 7 years ago

Comment on attachment 8827765 [details] [review]
github pull request

For every character I add or remove the height of the comment box grows!
Here is a short video of what I saw:

https://www.youtube.com/watch?v=2Qs_Wd7ezSk

	William Lachance (:wlach)
	Comment 6 • 7 years ago

This finally annoyed me enough that I created an updated PR using the autosizer library. :)

	Dylan Hardison [:dylan] (he/him)
	Comment 7 • 7 years ago

Comment on attachment 8884864 [details] [review]
PR using autosizer library

r-, per github comments:

Is there a way of making it readjust the size when the textarea changes? Currently if you reply to a long comment the box will not resize until you type stuff.

If that can't be fixed, let me know and I'll approve as is.

	William Lachance (:wlach)
	Comment 8 • 7 years ago

Comment on attachment 8884864 [details] [review]
PR using autosizer library

PR updated