Closed
Bug 98147
Opened 23 years ago
Closed 21 years ago
The 'ViewAll attachments' link should be greyed out if there are none
Categories
(Bugzilla :: Attachments & Requests, defect, P3)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: bugzilla, Assigned: goobix)
Details
Attachments
(1 file, 3 obsolete files)
1.48 KB,
patch
|
goobix
:
review+
|
Details | Diff | Splinter Review |
The 'ViewAll' link in the new attachments table should be unavailable if there are no links. And/Or if you click on it, and there are no attachments, then the http://bugzilla.mozilla.org/attachment.cgi?bugid=98074&action=viewall page should say 'there are no attachments to view', otherwise, you get a page that is potentially confusing, which says it is 'Viewing All Attachments', but there aren't actually any there.
Updated•23 years ago
|
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.18
Component: Creating/Changing Bugs → attachment and request management
Assignee | ||
Comment 1•21 years ago
|
||
Tested on my local Bugzilla installation and it works like expected.
Assignee | ||
Updated•21 years ago
|
Attachment #127922 -
Flags: review?(kiko)
Assignee | ||
Comment 2•21 years ago
|
||
Attachment #127922 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #127923 -
Flags: review?(kiko)
Assignee | ||
Comment 3•21 years ago
|
||
Attachment #127923 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #127922 -
Flags: review?(kiko) → review-
Assignee | ||
Updated•21 years ago
|
Attachment #127923 -
Flags: review?(kiko) → review-
Assignee | ||
Updated•21 years ago
|
Attachment #127924 -
Flags: review?(kiko)
Comment 4•21 years ago
|
||
Comment on attachment 127924 [details] [diff] [review] Removed != 0 condition compared to previous versions. r=kiko, nice work
Attachment #127924 -
Flags: review?(kiko) → review+
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Comment 6•21 years ago
|
||
I'm going to let Myk make the approval call on this one since it's a UI issue
Comment 7•21 years ago
|
||
Thanks for the patch. The coloring should be done with CSS rather than the font tag, but that's not a tragedy. The idea itself is a good one. a=myk
Flags: approval? → approval+
Comment 8•21 years ago
|
||
Err, sorry, I was looking at an earlier patch. Actually, I like the greying effect; it's standard for indicating that UI controls are disabled, so a patch that did that (via CSS) would be ideal if you have time for it (otherwise check this one in; I maintain my a=myk).
Assignee | ||
Comment 9•21 years ago
|
||
This should be ready for checkin then.
Attachment #127924 -
Attachment is obsolete: true
Assignee | ||
Comment 10•21 years ago
|
||
Comment on attachment 128012 [details] [diff] [review] New version per myk's suggestions. Setting kiko's r+ from the previous one.
Attachment #128012 -
Flags: review+
Comment 11•21 years ago
|
||
Kiko needs to explicitly say that his r= can be migrated to the newer attachment, but I've reviewed it, and it looks good, so the r= can stand, as does my a=. Go ahead and check it in if you have access, or I'll check it in if you don't.
Assignee | ||
Comment 12•21 years ago
|
||
I don't have access, so a checkin on this would be appreciated. Thanks for letting me know about the r+ transition policy.
Comment 13•21 years ago
|
||
Checking in css/edit_bug.css; /cvsroot/mozilla/webtools/bugzilla/css/edit_bug.css,v <-- edit_bug.css new revision: 1.3; previous revision: 1.2 done Checking in template/en/default/attachment/list.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/attachment/list.html.tmpl,v <-- list.html.tmpl new revision: 1.9; previous revision: 1.8 done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 14•21 years ago
|
||
For the record, I did say on IRC that he could carry over my r=.
Assignee | ||
Comment 15•21 years ago
|
||
I really don't know how that TAB got there. It's the 2nd time this is happening. I'm used with TAB environments, but I really fail to see how I could have pushed that key, especially after the line ended. Oh well, I'll switch completely to vim then, since it has support for auto-expanding TABs. Really sorry about it. :(
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•