Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove ability to resize textareas #1067

Closed
wants to merge 1 commit into from
Closed

remove ability to resize textareas #1067

wants to merge 1 commit into from

Conversation

morenoh149
Copy link

On android, without this line you see 3 diagonal lines in the lower right hand side of textareas. This suggests to the app user that the textarea is resizeable. On mobile apps you probably don't want the user to be able to resize textareas so I suggest this edit.

On android, without this line you see 3 diagonal lines in the lower right hand side of textareas. This suggests to the app user that the textarea is resizeable. On mobile apps you probably don't want the user to be able to resize textareas so I suggest this edit.
@ajoslin
Copy link
Contributor

ajoslin commented Apr 9, 2014

Adding a class to disable resizing of the textarea sounds best. People expect textareas to be resizable by default, I'd think.

@ajoslin
Copy link
Contributor

ajoslin commented Apr 9, 2014

Are you interested in opening a pull request for the class? Something like .no-resize

@morenoh149
Copy link
Author

depends on how opinionated ionic should be here. I don't think a user should EVER be able to increase a text area. But adding a class and documenting this behaviour would work too. Perhaps ping more core devs?

@adamdbradley adamdbradley self-assigned this Apr 10, 2014
@adamdbradley
Copy link
Contributor

I think we'll go with adding a new class because having the resizer by default is something most would expect. IMO, noticing that you do not like the resizer and having it explicitly removed, is better than expecting the resizer to be there, but not sure why it's not showing or how to get it to work.

517658e

@morenoh149 morenoh149 deleted the patch-1 branch May 1, 2014 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants