-
Notifications
You must be signed in to change notification settings - Fork 12
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
Ink/ally/text area update #189
Conversation
74dcfde
to
b79233a
Compare
@lukewestby Hey Luke! Tessa and I thought you might be a good person to review this PR given that there's custom element resizing stuff. |
@allymcknight Hello! First, one question: since this corrects a previously broken behavior without modifying any APIs, could the change be directly made to TextArea.V3 rather than adding a V4? If that were the case then it seems to me like we wouldn't have to consider the custom element. |
6e34263
to
961f2a0
Compare
@lukewestby sure! mind checking it out now? |
6aac998
to
71a74b0
Compare
@lukewestby ping! |
@allymcknight I think Luke is OOO until January 3. Maybe worth re-teambot-ing? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
FYI: the previous upgrade of noredink-ui is blocked in the monolith on this PR: https://github.com/NoRedInk/NoRedInk/pull/22436
This PR fixes a bug in the TextArea styling code which prevented the label from changing color upon text area focus.
OUTFOXES
https://www.pivotaltracker.com/story/show/162593210