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(ui): Make TextAreas the proper size #15634

Merged
merged 1 commit into from Oct 29, 2019

Conversation

hoorayimhelping
Copy link
Contributor

@hoorayimhelping hoorayimhelping commented Oct 29, 2019

Closes #15499

Apply clockface 1.0.3 to fix TextAreas.

Screen Shot 2019-10-29 at 11 13 27 AM
Screen Shot 2019-10-29 at 11 13 05 AM

@@ -23,6 +23,7 @@ module.exports = {
resolve: {
alias: {
src: path.resolve(__dirname, 'src'),
react: path.resolve('./node_modules/react'),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this'll let us do things like yarn add ../../clockface/dist/influxdata-clockface-1.0.3.tgz. to test local library builds in chrono 2

Copy link
Contributor

Choose a reason for hiding this comment

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

could you flesh out an example of this.. ? I was always npm linking clockface and giraffe. I should share how to do that with people now that I think about it.

@hoorayimhelping hoorayimhelping requested a review from a team October 29, 2019 18:20
@ghost ghost requested review from 121watts and ebb-tide and removed request for a team October 29, 2019 18:20
Copy link
Contributor

@ebb-tide ebb-tide left a comment

Choose a reason for hiding this comment

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

🍰

@@ -23,6 +23,7 @@ module.exports = {
resolve: {
alias: {
src: path.resolve(__dirname, 'src'),
react: path.resolve('./node_modules/react'),
Copy link
Contributor

Choose a reason for hiding this comment

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

could you flesh out an example of this.. ? I was always npm linking clockface and giraffe. I should share how to do that with people now that I think about it.

@hoorayimhelping hoorayimhelping merged commit 2498011 into master Oct 29, 2019
@hoorayimhelping hoorayimhelping deleted the bs_fix_squished_textareas branch October 29, 2019 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] Text Areas have improper heights
2 participants