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

Limit textarea resize within page/container border #8600

Merged

Conversation

sop-kim
Copy link
Contributor

@sop-kim sop-kim commented Dec 8, 2023

Closes #5210

UI fix

Technical

This change adds a resize field to textarea elements in the css to restrict the textarea from expanding beyond the page/container borders. For forms that can expand downwards, the value is vertical and for fixed size containers, the value is none.

Testing

Go to any page with a textarea element (eg. edit book) and try to resize it; it should only resize vertically for forms that can expand downwards, or not at all for containers that are a fixed size.

Pages tested:

  • books/edit/edition.html
  • books/edit/about.html
  • type/author/edit.html
  • type/list/edit.html
  • type/user/edit.html
  • macros/NotesModal.html

Not yet tested:

  • sectionopenlibrary\templates\admin\case.html
  • openlibrary\templates\books\edit\addfield.html
  • openlibrary\templates\edit_yaml.html
  • openlibrary\templates\lib\history.html
  • openlibrary\templates\type\about\edit.html
  • openlibrary\templates\type\doc\edit.html
  • openlibrary\templates\type\home\edit.html
  • openlibrary\templates\type\macro\edit.html
  • openlibrary\templates\type\permission\edit.html
  • openlibrary\templates\type\rawtext\edit.html
  • openlibrary\templates\type\template\edit.html
  • openlibrary\templates\type\user\edit.html
  • openlibrary\templates\type\usergroup\edit.html
  • openlibrary\macros\CreateListModal.html

Screenshot

Before:

image

After (can only resize vertically inside olform):

image

Stakeholders

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5af8db3) 17.61% compared to head (bf3a2bb) 17.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8600   +/-   ##
=======================================
  Coverage   17.61%   17.61%           
=======================================
  Files          85       85           
  Lines        4456     4456           
  Branches      782      782           
=======================================
  Hits          785      785           
  Misses       3187     3187           
  Partials      484      484           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@scottbarnes scottbarnes left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks, @sop-kim!

@scottbarnes scottbarnes merged commit 5643463 into internetarchive:master Dec 18, 2023
3 checks passed
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.

Textarea resize beyond page border
3 participants