Closed
Bug 841490
Opened 12 years ago
Closed 6 years ago
Show a glimpse of a comment as a tool-tip when linkifying "bug X comment Y"
Categories
(Bugzilla :: User Interface, enhancement)
Bugzilla
User Interface
Tracking
()
RESOLVED
DUPLICATE
of bug 1146763
People
(Reporter: koosha.khajeh, Unassigned)
References
(Depends on 1 open bug)
Details
Currently, bug 800000 and bug 800000 comment 2 both show the bug's description and status. It would be nice if bug 800000 comment 2 show instead a sub-string of comment 2 as a the tool-tip of the hyperlink. This way, the comment could be read or recalled without opening the mentioned bug.
Comment 1•8 years ago
|
||
I see this issue is still relevant, can I go on and implement it?
Flags: needinfo?(koosha.khajeh)
Comment 2•8 years ago
|
||
This task does not have an assignee set, so anyone can propose a patch.
Flags: needinfo?(koosha.khajeh)
Comment 3•8 years ago
|
||
(In reply to Andre Klapper from comment #2)
> This task does not have an assignee set, so anyone can propose a patch.
okay great, I'll try sending a PR on github then. Will report back updates here.
Comment 4•8 years ago
|
||
Okay, so this needs a few things:
So each bug link, including ones to comments is
https://github.com/bugzilla/bugzilla/blob/c67b3b303adba379f3625aa88a61d5fb49bd319d/template/en/default/bug/link.html.tmpl
the template variable comment_num will be true for a link to a bug comment.
The first part would be putting that info into some data attributes.
Now we have some choices to make... do we really want to load a blurb
for each referenced comment? or do we want to lazy-load them in JS?
For the later, we'd add the bug id and comment number to data- attributes on the a.bz_bug_link elements,
and need to add an API that can get comment blurbs.
Note that this also require adding an internal API for getting "comment N of bug B" but I am more than happy to implement that part. :-)
Comment 5•6 years ago
|
||
Can be solved along with other hovercards.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•