Skip to content

WP-1208: Fix 'Show More' view display results for long descriptions#1271

Open
fnets wants to merge 3 commits intomainfrom
task/WP-1208
Open

WP-1208: Fix 'Show More' view display results for long descriptions#1271
fnets wants to merge 3 commits intomainfrom
task/WP-1208

Conversation

@fnets
Copy link
Copy Markdown
Contributor

@fnets fnets commented Apr 8, 2026

Overview

If the shared workspace description is too long and Show More is clicked, it takes up the entire screen and unable to scroll

Related

Changes

Addes overflow to the CSS for the description class.

Testing

  1. Either create a shared workspace with a very long decsription (around 500 chars), or go here: https://cep.test/workbench/data/tapis/projects/cep.project.CEP-1892
  2. Click the Show More button on the description.
  3. Verify that you are able to scroll down and read the whole description and get to the Show Less button.

UI

Screenshot 2026-04-08 at 3 58 13 PM

Notes

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.86%. Comparing base (74f4b1c) to head (d0125e3).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1271   +/-   ##
=======================================
  Coverage   63.86%   63.86%           
=======================================
  Files         451      451           
  Lines       13481    13481           
  Branches     2658     2658           
=======================================
  Hits         8610     8610           
  Misses       4532     4532           
  Partials      339      339           
Flag Coverage Δ
javascript 66.88% <ø> (ø)
unittests 60.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@edmondsgarrett edmondsgarrett left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@shayanaijaz shayanaijaz left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants