Skip to content

Conversation

@urmauur
Copy link
Member

@urmauur urmauur commented Sep 26, 2025

Describe Your Changes

21899

Fixes Issues

Screenshot 2025-09-26 at 14 58 32
  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Important

Improves handling of long project titles in ProjectContent by truncating with ellipsis and adding hover tooltips.

  • UI Changes:
    • In ProjectContent function in index.tsx, added truncate class and title attribute to <h3> element for project names to handle long titles.
    • Moved thread count display to a separate <span> element with shrink-0 and whitespace-nowrap classes for better alignment.
  • Behavior:
    • Long project titles are now truncated with ellipsis and full title is shown on hover via title attribute.

This description was created by Ellipsis for 3d224f8. You can customize this summary. It will automatically update as commits are pushed.

@urmauur urmauur added this to the v0.7.0 milestone Sep 26, 2025
@urmauur urmauur requested a review from Minh141120 September 26, 2025 08:00
@urmauur urmauur self-assigned this Sep 26, 2025
@urmauur urmauur added this to Jan Sep 26, 2025
@urmauur urmauur moved this to Needs Review in Jan Sep 26, 2025
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 3d224f8 in 53 seconds. Click for details.
  • Reviewed 51 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. web-app/src/routes/project/index.tsx:137
  • Draft comment:
    Good fix: Adding 'min-w-0' to the flex containers ensures that long project names are properly truncated in the layout.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. web-app/src/routes/project/index.tsx:148
  • Draft comment:
    Nice update: Switching from 'line-clamp-1' to 'truncate' in the h3 with an added title attribute improves text overflow handling and accessibility.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. web-app/src/routes/project/index.tsx:161
  • Draft comment:
    Moving the thread count span to a separate flex item with 'shrink-0 whitespace-nowrap' helps maintain layout consistency even when the project name is long.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_qigTeKDCJPb936pM

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@github-actions
Copy link
Contributor

Barecheck - Code coverage report

Total: 29.73%

Your code coverage diff: 0.00% ▴

Uncovered files and lines
FileLines
web-app/src/routes/project/index.tsx1-2, 4-6, 8-10, 18-20, 22, 24-26, 28-30, 32-43, 45-48, 50-56, 58-66, 68-70, 73-79, 81-91, 93-104, 106-122, 124-130, 132-135, 137-149, 151-173, 175, 177-181, 183-186, 188, 190-196, 198-206, 208-214, 217-223, 225, 227-228, 230-245, 247

Copy link
Member

@Minh141120 Minh141120 left a comment

Choose a reason for hiding this comment

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

LGTM!

@urmauur urmauur merged commit 580bdc5 into dev Sep 26, 2025
21 checks passed
@urmauur urmauur deleted the chore/projects branch September 26, 2025 08:47
@github-project-automation github-project-automation bot moved this from Needs Review to QA in Jan Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants