Skip to content
This repository was archived by the owner on Nov 25, 2020. It is now read-only.

inputview: implement autogrow for single line inputs#155

Merged
gokmen merged 3 commits intokoding:masterfrom
sinan:master
Apr 28, 2016
Merged

inputview: implement autogrow for single line inputs#155
gokmen merged 3 commits intokoding:masterfrom
sinan:master

Conversation

@sinan
Copy link
Copy Markdown
Member

@sinan sinan commented Apr 28, 2016

cc/ @usirin

@sinan
Copy link
Copy Markdown
Member Author

sinan commented Apr 28, 2016

gif

@gokmen gokmen merged commit 9a035b9 into koding:master Apr 28, 2016
@_clone = $ '<div/>', { class: 'invisible' }

{ type } = @getOptions()
isVertical = type.toLowerCase() is 'textarea'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not accepting this as an option?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't it an option already?

Copy link
Copy Markdown
Contributor

@usirin usirin Apr 28, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

operation we are making is named isVertical, but the option we are passing is textarea. Can't an input/textarea autogrow both vertically and horizontally?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assumption we did earlier was to use autogrow only for textareas and just vertically, now we expand the functionality to text inputs basically, but they can only grow horizontally. That's why I didn't think another option was necessary.

However I didn't think about a bidirectional growing input, never needed that. Do you think it is necessary?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, I don't think so. Only the assumption we make made me confused.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants