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

Fix text-area scroll with fixed height in material theme #320

Closed
wants to merge 1 commit into from

Conversation

niiyeboah
Copy link

@niiyeboah niiyeboah commented Feb 15, 2019

Fixes #310


This change is Reviewable

@CLAassistant
Copy link

CLAassistant commented Feb 15, 2019

CLA assistant check
All committers have signed the CLA.

@niiyeboah niiyeboah force-pushed the fix/text-area-scroll-material#310 branch from cbd463e to 4710d0b Compare February 15, 2019 12:38
@samiheikki
Copy link
Contributor

With this fix the error label is displayed above the material line when it should be under it.

height-max

@niiyeboah niiyeboah force-pushed the fix/text-area-scroll-material#310 branch 2 times, most recently from af71224 to c0d4a13 Compare February 25, 2019 16:17
Copy link
Member

@web-padawan web-padawan left a comment

Choose a reason for hiding this comment

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

Let's try to avoid DOM structure changes, as it would be probably a breaking one.

@tomivirkki
Copy link
Member

This could probably be fixed by moving the underline from [part="input-field"] to .vaadin-text-area-container

@niiyeboah
Copy link
Author

This could probably be fixed by moving the underline from [part="input-field"] to .vaadin-text-area-container

That will cause the same issue shown in the screenshot @samiheikki posted. The red underline is underneath the error message.

@niiyeboah niiyeboah force-pushed the fix/text-area-scroll-material#310 branch from c0d4a13 to 2712ac4 Compare February 27, 2019 15:54
@jtomass jtomass added this to 🏗  In Progress in vaadin-core Mar 29, 2019
@web-padawan web-padawan moved this from 🏗  In Progress to In Review (unscoped) in vaadin-core Apr 1, 2019
@tomivirkki tomivirkki closed this Jul 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
vaadin-core
  
In Review (unscoped)
Development

Successfully merging this pull request may close these issues.

[text-field] Broken vaadin-text-area scroll in Material theme
5 participants