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

[EnhancedTextArea] add shadowStyle #3274

Closed
wants to merge 1 commit into from
Closed

Conversation

cgestes
Copy link
Contributor

@cgestes cgestes commented Feb 10, 2016

(in my case to allow applying box-sizing)

Also added border: 'none' to make it more like the visible one. (2 pixels does not change a lot... but you never know)

@alitaheri alitaheri added the PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI label Feb 10, 2016
@alitaheri
Copy link
Member

@cgestes Please rebase your work onto #3252. To avoid conflicts.

@cgestes cgestes changed the title [EnhancedTextArea] allow style shadow [EnhancedTextArea] add shadowStyle Feb 10, 2016
(in my case to allow applying box-sizing)
@cgestes
Copy link
Contributor Author

cgestes commented Feb 10, 2016

will wait till #3252 is merged.

@alitaheri
Copy link
Member

Or that 😆 😆

@cgestes
Copy link
Contributor Author

cgestes commented Feb 10, 2016

Should we mirror the style of the textarea and just override the important properties instead?

Otherwize the shadow has to be adjusted by hand which is very error prone

@cgestes
Copy link
Contributor Author

cgestes commented Feb 10, 2016

too much related commits. making only one PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants