Closed
Bug 1057347
Opened 10 years ago
Closed 10 years ago
logviewer: Copying log lines copies line numbers too
Categories
(Tree Management :: Treeherder, defect, P2)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: automatedtester, Mentored)
References
()
Details
(Whiteboard: [good first bug])
Attachments
(1 file)
1) Open the structured log viewer for a failed job
2) Jump to the failure by clicking it in the overview
3) Copy multiple log lines, with a bit of context before/after the failure line
4) Paste somewhere
Expected:
Pasted text is the raw log
Actual:
Pasted text is the raw log with a log viewer line number inserted at the start of each line.
It's a very frequent workflow to copy parts of a log into a bug.
This makes the logs harder to read when pasted into a bug, and doesn't match TBPL behaviour.
Comment 1•10 years ago
|
||
I looked at this briefly. While you can make the line number unselectable, it appears it still ends up in the copy buffer. From what I've read so far on stack overflow and elsewhere, this appears to be the way selection functionality works, it copies everything in the container(s), even when they aren't displayed as selected.
Caveat I am a newbie :) That being said, the TBPL brief-log or full-log don't appear to me to have any line numbers at all. Which I gather is why the problem wasn't encountered. We could potentially remove the line numbers completely from the treeherder structured log, if copy/paste from structured is an impediment to adoption.
Another option perhaps is the bug could be marked as wontfix while this behavior remains missing - and the workaround is the user simply clicks on the raw log icon (either from the job panel or the structure log), and copies the equivalent chunk there.
Thoughts?
Comment 2•10 years ago
|
||
Actually, when I think about it, clearly Github itself is doing this successfully when viewing code. You can copy and paste lines of code and they somehow prevent the line numbers' copy to the buffer and also keep the lines aligned during scroll. If we want this, perhaps we can look to their implementation.
Updated•10 years ago
|
Updated•10 years ago
|
Assignee | ||
Comment 3•10 years ago
|
||
Updated•10 years ago
|
Attachment #8488334 -
Flags: review?(cdawson) → review+
Comment 4•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder-ui
https://github.com/mozilla/treeherder-ui/commit/b2fe04392b8f549fcefe063a2b56b321608cef44
Bug 1057347: Remove log line number from logviewer to allow easier copy/paste of logs
https://github.com/mozilla/treeherder-ui/commit/8cc9bce38cbc3f0312408870a09297854f8ec7b0
Merge pull request #175 from AutomatedTester/1057347
Bug 1057347: Remove log line number from logviewer to allow easier copy/paste of logs
Reporter | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 5•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/b0522ec50ac5fc65f4e8a2fa3954e672504f55c4
Bug 1057347: Remove log line number from logviewer to allow easier copy/paste of logs
https://github.com/mozilla/treeherder/commit/c6e4178f0884e0013157ae2bc063862d33eed26d
Merge pull request #175 from AutomatedTester/1057347
Bug 1057347: Remove log line number from logviewer to allow easier copy/paste of logs
You need to log in
before you can comment on or make changes to this bug.
Description
•