Edgewall Software
Modify

Opened 11 years ago

Last modified 7 months ago

#11125 new enhancement

Truncate very long text in the ticket properties dialog

Reported by: Ryan J Ollos Owned by:
Priority: normal Milestone: next-stable-1.6.x
Component: ticket system Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Use a bit of CSS to prevent this from happening. It's a bit of a corner case because the text would have to be quite long, but will be easy to fix.

Attachments (5)

Overspill.png (15.8 KB ) - added by Ryan J Ollos <ryan.j.ollos@…> 11 years ago.
BreakAll.png (18.4 KB ) - added by Ryan J Ollos <ryan.j.ollos@…> 11 years ago.
NoWrapping.png (17.6 KB ) - added by Ryan J Ollos <ryan.j.ollos@…> 11 years ago.
AllowWrapping.png (17.9 KB ) - added by Ryan J Ollos <ryan.j.ollos@…> 11 years ago.
LongAuthorField.png (93.4 KB ) - added by Ryan J Ollos 9 years ago.

Download all attachments as: .zip

Change History (14)

by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

Attachment: Overspill.png added

by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

Attachment: BreakAll.png added

by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

Attachment: NoWrapping.png added

by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

Attachment: AllowWrapping.png added

comment:1 by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

I see 3 options here.

  1. Truncate long text that doesn't break normally.
  2. Don't truncate, rather force words to break.
  3. Force all text to truncate and don't break any text.

  • trac/htdocs/css/ticket.css

    diff --git a/trac/htdocs/css/ticket.css b/trac/htdocs/css/ticket.css
    index c482dee..460a6b3 100644
    a b  
    101101 width: 20%;
    102102}
    103103#ticket table.properties th.missing { color: #d3d398; }
    104 #ticket table.properties td { width: 30% }
     104#ticket table.properties td {
     105 overflow: hidden;
     106 text-overflow: ellipsis;
     107 width: 30%;
     108}
    105109#ticket table.properties td p:first-child { margin-top: 0 }
    106110#ticket table.properties td p:last-child { margin-bottom: 0 }
    107111#ticket table.properties .description { border-top: 1px solid #dd9 }

  • trac/htdocs/css/ticket.css

    diff --git a/trac/htdocs/css/ticket.css b/trac/htdocs/css/ticket.css
    index c482dee..804cb24 100644
    a b  
    101101 width: 20%;
    102102}
    103103#ticket table.properties th.missing { color: #d3d398; }
    104 #ticket table.properties td { width: 30% }
     104#ticket table.properties td {
     105 width: 30%;
     106 word-break: break-all;
     107}
    105108#ticket table.properties td p:first-child { margin-top: 0 }
    106109#ticket table.properties td p:last-child { margin-bottom: 0 }
    107110#ticket table.properties .description { border-top: 1px solid #dd9 }

  • trac/htdocs/css/ticket.css

    diff --git a/trac/htdocs/css/ticket.css b/trac/htdocs/css/ticket.css
    index c482dee..f508812 100644
    a b  
    101101 width: 20%;
    102102}
    103103#ticket table.properties th.missing { color: #d3d398; }
    104 #ticket table.properties td { width: 30% }
     104#ticket table.properties td {
     105 overflow: hidden;
     106 text-overflow: ellipsis;
     107 width: 30%;
     108 white-space: nowrap;
     109}
    105110#ticket table.properties td p:first-child { margin-top: 0 }
    106111#ticket table.properties td p:last-child { margin-bottom: 0 }
    107112#ticket table.properties .description { border-top: 1px solid #dd9 }
Last edited 11 years ago by Christian Boos (previous) (diff)

comment:2 by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

Btw, my preference is (3), since there is plenty of room for text of reasonable length and enough characters to infer the actual name even in the case of truncation.

comment:3 by Christian Boos, 11 years ago

Milestone: next-stable-1.0.x

With the white-space: nowrap in 3), you'd not see all the people in CC:, you'd not see all the keywords, etc. I think it's not OK.

Then between 1) and 2), I think I prefer 2), in order to not lose information. Otherwise the only place in the page where you could see the full content for the field is where it has been set in the change history.

comment:4 by Ryan J Ollos <ryan.j.ollos@…>, 11 years ago

I hadn't given this enough thought and considered CC and keywords, so I would agree that (3) is not okay. As to (1) or (2) then, I have no preference. If we truncated, we could show the full name on hover-over, but you are probably right that it's better to just not truncate.

comment:5 by Ryan J Ollos, 10 years ago

Reporter: changed from Ryan J Ollos <ryan.j.ollos@…> to Ryan J Ollos

by Ryan J Ollos, 9 years ago

Attachment: LongAuthorField.png added

comment:6 by Ryan J Ollos, 9 years ago

We should also deal with the possibility of very long text in the author field of a comment. Here is a screen capture from t-h.o:

comment:7 by Ryan J Ollos, 7 years ago

Milestone: next-stable-1.0.xnext-stable-1.2.x

Moved ticket assigned to next-stable-1.0.x since maintenance of 1.0.x is coming to a close. Please move the ticket back if it's critical to fix on 1.0.x.

comment:8 by Ryan J Ollos, 4 years ago

Milestone: next-stable-1.2.xnext-stable-1.4.x

comment:9 by Ryan J Ollos, 7 months ago

Milestone: next-stable-1.4.xnext-stable-1.6.x

Milestone renamed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.