-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Disable resizing the textarea in CollectionCreate
modal
#2974
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
Conversation
@@ -75,6 +75,7 @@ | |||
|
|||
.form__input-flexible-height { | |||
height: auto; | |||
resize: none; |
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.
🚀
@@ -24,10 +24,9 @@ | |||
display: flex; | |||
flex-wrap: wrap; | |||
flex-flow: column; | |||
max-height: 80%; |
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.
This is the style for ALL modal, not just "create collection", so we need to tread carefully here. There are some larger modals like "keyboard shortcuts" where the max-height
is required. It makes it so that the modal itself is limited in height and forces the content section of the modal to have a scrollbar if the contents won't fit.
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.
I apologize for the oversight, I failed to consider the potential utilization of this class for other modals.
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.
@lindapaiste, we can solve this issue using the :has()
pseudo-class. I have made the commit you can check it.
CollectionCreate
modal
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.
Thanks for your work on this, I think this looks good to me! I'm not sure how many folks currently utilize resizing the textbox, but we can see if anyone misses it down the line!
Fixes #2973
Following changes are made in this pr :-
max-width
which was causing the overflowing of the 'Create collection' button.min-width
property when the screen width is650px
or less.I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.