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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: mkanat, Assigned: dkl)

References

Details

Attachments

(1 file, 1 obsolete file)

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
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
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
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)
(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
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.
(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
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 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-
Agreed, let's just remove it.
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 on attachment 578044 [details] [diff] [review] Patch to revert to old bug link (v2) r=LpSolit
Attachment #578044 - Flags: review?(LpSolit) → review+
Flags: approval+
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.

Attachment

General

Created:
Updated:
Size: