Closed
Bug 532478
Opened 16 years ago
Closed 13 years ago
When attaching a file to a bug, display the max size allowed for attachments
Categories
(Bugzilla :: Attachments & Requests, enhancement)
Bugzilla
Attachments & Requests
Tracking
()
RESOLVED
FIXED
Bugzilla 4.4
People
(Reporter: bluedzins, Assigned: sjoshi)
References
()
Details
(Keywords: ue)
Attachments
(1 file)
|
931 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (compatible; Konqueror/3.5; Linux 2.6.27.37-0.1-default; X11; x86_64) KHTML/3.5.10 (like Gecko) SUSE
Build Identifier:
Originated from:
https://bugzilla.novell.com/show_bug.cgi?id=559058
It is better when explained in examples what is wrong.
100MB file of attachment. I transfer such file and and the end I get the info
saying that sorry, but limit for attachment is 1MB.
The wished UI -- near the file entry in adding attachment page please show
label "Please note: the limit for file is XXX MB". This way I don't have to
upload 100MB just to know it is too much.
Of course if it would be possible to check the size, and the decide to upload the file it would be the best.
Usability bug -- I am setting it to minor, despite how annoying it is for users with poor connection.
Reproducible: Always
Comment 1•16 years ago
|
||
(In reply to comment #0)
> Of course if it would be possible to check the size, and the decide to upload
> the file it would be the best.
You cannot check the size of the file before it's uploaded, so this wouldn't work. But the limit note is a good idea.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86 → All
Summary: [JJ] adding attachment: show the info about the limits in advance → Adding attachment: show the info about the limits in advance
| Assignee | ||
Comment 2•13 years ago
|
||
Implemented file size limit note on attachment.cgi.
Attachment #666613 -
Flags: review?(LpSolit)
Comment 3•13 years ago
|
||
Comment on attachment 666613 [details] [diff] [review]
Patch
>+ <br><em>(File size limit:[% max_limit FILTER html %] KB)</em>
Missing whitespace after colons. r=LpSolit
Attachment #666613 -
Flags: review?(LpSolit) → review+
Updated•13 years ago
|
Assignee: attach-and-request → joshi_sunil
Severity: minor → enhancement
Status: NEW → ASSIGNED
Flags: approval4.4+
Flags: approval+
Target Milestone: --- → Bugzilla 4.4
Comment 4•13 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/attachment/createformcontents.html.tmpl
Committed revision 8430.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified template/en/default/attachment/createformcontents.html.tmpl
Committed revision 8420.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Summary: Adding attachment: show the info about the limits in advance → When attaching a file to a bug, display the max size allowed for attachments
You need to log in
before you can comment on or make changes to this bug.
Description
•