It's a hassle to be reading comments and then need to scroll back to the top of the page to be able to edit an attachment. A simple edit link next to the attachment name would improve usability. Patch coming up.
Created attachment 153849 [details] [diff] [review] kiko_v1: add edit link See http://www.async.com.br/~kiko/mybugzilla/show_bug.cgi?id=3 for example.
Glob suggested following [reply] and adding square brackets; I don't care either way (and I want to change [reply] to be in a table cell and aligned consistently) but I can change it while checking in if approved.
Status: NEW → ASSIGNED
Interesting... this is going to add the (edit) link after any "attachment ####" that gets linkified in a comment, too. Was that intentional? Myk, how's your take on it? (UI change)
Created attachment 153914 [details] [diff] [review] option 2: only add (edit) to attachment creation link. That was actually unintentional. I'm not sure which is more correct, because I don't have a lot of experience with observing people using attachment links (they are rarely used outside Bugzilla IME), so I'm attaching this one, and feel free to approve any one of them you like.
Comment on attachment 153914 [details] [diff] [review] option 2: only add (edit) to attachment creation link. See what you think of either one.
I suspect that a better solution is to make the attachment link itself go to the edit page. From there the other attachment views (raw and diff) are one click away, albeit not very noticably (but that can be fixed in another bug). This would be a change to longstanding functionality, so people may complain, but others may praise, and it's worth it if the majority of people want the edit page when they click an attachment link.
(In reply to comment #6) > I suspect that a better solution is to make the attachment link itself go to > the edit page. But an "(edit)" link is a good solution for the iterim at least and gives us a chance to see what people think. Version one that puts the link everywhere approved.
Flags: approval? → approval+
Created attachment 154160 [details] [diff] [review] kiko_v2: remove view, unnecessary and inconvenient for shell pasting
This simple, low-risk usability fix should go into 2.18.
Checked into branch and trunk. Thanks!
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Question: how do I localize the word "edit", which we're now inserting in comment bodies after attachment links....? I understand grandfathering existing hard-coded stuff (like "attachment" and "bug") in quoteUrls, but weren't we trying to avoid adding new non-localizable content?
The only way I see we could do it is providing a template block that would be processed every time an attachment or bug was encountered. I'm not sure if that would hurt us performance-wise, but I'd be happy to implement this for our Get*Link functions if it's decided it's how it should be.
Given that we start off by searching for the string 'attachment' in the first place, I'm not sure that localising 'edit' is really the top priority here...
(In reply to comment #13) > Given that we start off by searching for the string 'attachment' in the first > place, I'm not sure that localising 'edit' is really the top priority here... Brad has a point here. This can be tackled when we do i18n for GetBugLink :)
OS: Linux → All
Hardware: PC → All
Target Milestone: --- → Bugzilla 2.18
You need to log in before you can comment on or make changes to this bug.