8000 Fixed issue #5512 #modxbughunt by devlanda · Pull Request #13524 · modxcms/revolution · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fixed issue #5512 #modxbughunt #13524

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

Closed
wants to merge 1 commit into from
Closed

Fixed issue #5512 #modxbughunt #13524

wants to merge 1 commit into from

Conversation

devlanda
Copy link
@devlanda devlanda commented Jul 7, 2017

What does it do?

It makes it possible to resize the textarea (input type TV) in vertical direction. The css code adds the resize option on the right bottom edge. So you don't have to scroll when you have more than 7 lines.

Why is it needed?

Currently you have to scroll when you have above 7 lines in a textarea. In that case you may loose the overview.

Related issue(s)/PR(s)

TV input type "textarea" not high enough (#5512)

@rtripault rtripault added this to the v2.5.8 milestone Jul 7, 2017
@rtripault rtripault added the type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript. label Jul 7, 2017
@gpsietzema
Copy link
Contributor

#modxbughunt #3points to @devlanda

@OptimusCrime
Copy link
Contributor

Have you tested that this actually works? I am pretty sure ExtJS and all its weird styling was the reason this feature was not added when the original issue was opened.

@devlanda
Copy link
Author
devlanda commented Jul 7, 2017

You are right! I did the test in browser. I was unaware about the weird impact of ExtJS.
This fix can be rejected.

@devlanda devlanda closed this Jul 7, 2017
@OptimusCrime
Copy link
Contributor

No problem. Just had a feeling ExtJS did not play along that nicely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0