Closed
Bug 703501
Opened 13 years ago
Closed 13 years ago
Linking to comment in top link is not useful for the common workflow of editing fields
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.4
People
(Reporter: mkanat, Assigned: dkl)
References
Details
Attachments
(1 file, 1 obsolete file)
502 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
When I click on the link at the top of a bugmail, it's nearly always to see/change fields, and not to comment. Even if it were to comment, my comment box stands normally at the top of the comments, so scrolling up to it is annoying. I'd really rather just have the link go to the bug itself, and let me make up my mind what I'm going to do on the page.
this isn't a bmo specific customisation - bug 365709 and bug 697371
Assignee: nobody → email-notifications
Component: General → Email Notifications
Product: bugzilla.mozilla.org → Bugzilla
QA Contact: general → default-qa
Version: Current → 4.3
Reporter | ||
Comment 2•13 years ago
|
||
Oh wow. That definitely needs to be reverted. If the link in comments is a problem, then just get rid of it. It wasn't a hugely-asked-for feature, and when we implemented it, it just caused trouble.
Flags: blocking4.2+
Target Milestone: --- → Bugzilla 4.2
Comment 3•13 years ago
|
||
Not a blocker. This code didn't land in 4.2. 5.0 only. As I said already, we could offer two links, so that everybody is free to choose the one he needs:
https://bugzilla.mozilla.org/show_bug.cgi?id=422256 (this bug)
https://bugzilla.mozilla.org/show_bug.cgi?id=422256#c2 (this comment)
Personally, my workflow is the same as mkanat, i.e. I'm more interested in bug fields than the bug comment. So IMO, we either propose both links, or we remove the one pointing to the comment. But let's not have a 4th bug about this topic!
Flags: blocking4.2+
Target Milestone: Bugzilla 4.2 → Bugzilla 5.0
Comment 4•13 years ago
|
||
Did someone backport this for bmo or something? This is currently live on bmo and we're certainly not even running 4.2 yet (still 4.0.2)
Assignee | ||
Comment 5•13 years ago
|
||
(In reply to Dave Miller [:justdave] from comment #4)
> Did someone backport this for bmo or something? This is currently live on
> bmo and we're certainly not even running 4.2 yet (still 4.0.2)
Yes. It was a simple change so we added it to bmo/4.0 and bmo/4.2.
dkl
Reporter | ||
Comment 6•13 years ago
|
||
Yeah, I think two links is too much space to take up for too little value. Let's just remove this feature from the text email.
Assignee | ||
Comment 7•13 years ago
|
||
(In reply to Max Kanat-Alexander from comment #6)
> Yeah, I think two links is too much space to take up for too little value.
> Let's just remove this feature from the text email.
I am not opposed to putting it back similar to what I had not too long ago. Where the main link at the top of the email was to the bug itself. And then have a separate link to the actual comment near the comment itself in the email. It was felt that it was too cluttered or that people thought the link was part of the comment text. If we can come up with a way to make the link look separate from the comment text, I still feel that it is useful to be there somewhere.
dkl
Assignee | ||
Comment 8•13 years ago
|
||
Patch that puts old bug link back to the way it was and shows a link near the comment to the bug comment.
dkl
Assignee: email-notifications → dkl
Status: NEW → ASSIGNED
Attachment #577702 -
Flags: review?(mkanat)
Comment 9•13 years ago
|
||
Comment on attachment 577702 [details] [diff] [review]
Patch to revert bug old bug link and add comment link with comments (v1)
> --- Comment #[% comment.count %] from [% comment.author.identity %] ---
>+--- [% urlbase %]show_bug.cgi?id=[% bug.id %]#c[% comment.count FILTER none %]
No, you are reimplementing bug 365709, which is why bug 697371 was then filed and finally this bug. We don't want any link in the comment itself as it's really too confusing. Let's remove it. That was a bad idea.
Attachment #577702 -
Flags: review?(mkanat) → review-
Reporter | ||
Comment 10•13 years ago
|
||
Agreed, let's just remove it.
Assignee | ||
Comment 11•13 years ago
|
||
Patch that returns text bugmail to old style of showing buglink with out link to comment.
dkl
Attachment #577702 -
Attachment is obsolete: true
Attachment #578044 -
Flags: review?(LpSolit)
Comment 12•13 years ago
|
||
Comment on attachment 578044 [details] [diff] [review]
Patch to revert to old bug link (v2)
r=LpSolit
Attachment #578044 -
Flags: review?(LpSolit) → review+
Updated•13 years ago
|
Flags: approval+
Assignee | ||
Comment 13•13 years ago
|
||
trunk:
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bugzilla/trunk
modified template/en/default/email/bugmail.txt.tmpl
Committed revision 8025
bmo/4.0:
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.0
modified template/en/default/email/newchangedmail.txt.tmpl
Committed revision 7974
bmo/4.2:
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2
modified template/en/default/email/bugmail.txt.tmpl
Committed revision 7971.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•