label-studio icon indicating copy to clipboard operation
label-studio copied to clipboard

fix: LEAP-1581: Improve styles in new comments

Open hlomzik opened this issue 1 year ago • 2 comments

There is a handy -webkit-line-clamp rule! All items are inlined to be just a text, but truncated after 3 lines, so should be good for a lot of different content.

Screenshot 2024-10-17 at 15 33 49

Also

Scales down the comment icon to always fit it inside usual editor screen with paddings. Fixes LEAP-1578 Style fixes for comment's classification after LEAP-1476

Screenshot 2024-10-17 at 15 34 59

PR fulfills these requirements

  • [ ] Tests for the changes have been added/updated (for bug fixes/features)
  • [ ] Docs have been added/updated (for bug fixes/features)
  • [ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
  • [x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

  • [ ] Product design
  • [ ] Backend (Database)
  • [ ] Backend (API)
  • [x] Frontend

Does this PR introduce a breaking change?

  • [ ] Yes, and covered entirely by feature flag(s)
  • [ ] Yes, and covered partially by feature flag(s)
  • [x] No
  • [ ] Not sure (briefly explain the situation below)

hlomzik avatar Oct 17 '24 13:10 hlomzik

Deploy Preview for heartex-docs canceled.

Name Link
Latest commit ddddd3c65c72b3bd213c44ca8173df4788095e9b
Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/6711207135a5df0008aa36ef

netlify[bot] avatar Oct 17 '24 13:10 netlify[bot]

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
Latest commit ddddd3c65c72b3bd213c44ca8173df4788095e9b
Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/6711207146c1570008757987

netlify[bot] avatar Oct 17 '24 13:10 netlify[bot]

Follow Merge downstream workflow has been failed.

Workflow run

robot-ci-heartex avatar Oct 22 '24 17:10 robot-ci-heartex