Closed
Bug 1290326
Opened 9 years ago
Closed 9 years ago
Wrap signatures in Top Crasher lists, rather than truncating them
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: n.nethercote, Assigned: adrian)
Details
Attachments
(5 files)
In Top Crasher lists, signatures get truncated, as the attached screenshot shows. Signatures can be quite long and often the truncation omits helpful detail.
(For example, I was just pestering lonnen on IRC because I couldn't find some crashes that I expected to see in the Top Crasher list. Turns out I was searching for a substring in the part of the signature that had been truncated.)
In other places the signatures are wrapped instead of truncated, e.g. in super search results. It would nice if this occurred in Top Crasher results as well.
Comment 1•9 years ago
|
||
Instead of wrapping we ought to use css overflow.
![]() |
Reporter | |
Comment 2•9 years ago
|
||
> Instead of wrapping we ought to use css overflow.
Which value? It needs to support searching in the overflowing part.
Comment 3•9 years ago
|
||
So that was interesting. Here I have a prototype where the `td.signature-column` has a `overflow:hidden` etc.
One of the top signatures contains the string "PNecko" but it's not visible.
When I start typing it finds it but it's not highlighted so I don't know where it is. In fact, it's kinda useless to find out that the text exists (but hidden).
Tried in Chrome. It's no better there.
Comment 4•9 years ago
|
||
This is what happens when you don't truncate the signatures at all. See how the table can't be compressed any more and you get horizontal scrolling.
This is full-screen here on my 15" macbook.
The bestest solution is probably to NOT truncate AND to remove some columns. Like Correlations.
We could also consider putting multiple pieces of data in a shared column. E.g. Combine "Rank", "%" and "Diff" into "Rank/%/Diff" but then you can't sort by Diff. :(
Or combine Signature and Bugzilla IDS like this "<th>Signature<br>Bugzilla IDs</th>".
What do you think?
Comment 5•9 years ago
|
||
Adrian probably has better ideas and opinions here.
![]() |
Reporter | |
Comment 6•9 years ago
|
||
Super Search results wrap the signature nicely, as this screenshot shows. Can you replicate that?
Assignee | ||
Comment 7•9 years ago
|
||
Assignee: nobody → adrian
Comment 8•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/902ec79a38245fa4da066208a1ebf74f9661f76a
Fixes bug 1290326 - Do not truncate signatures in Topcrashers. (#3418)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 9•9 years ago
|
||
Yay! This went to prod seconds ago. Oh why didn't we do this earlier?! :)
![]() |
Reporter | |
Comment 10•9 years ago
|
||
\o/
You need to log in
before you can comment on or make changes to this bug.
Description
•