Closed
Bug 182396
Opened 22 years ago
Closed 14 years ago
Parens around requestee field should be removed
Categories
(Bugzilla :: User Interface, defect)
Bugzilla
User Interface
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: timeless, Assigned: guy.pyrzak)
References
Details
(Whiteboard: [blocker will fix])
Attachments
(1 file)
|
1.67 KB,
patch
|
goobix
:
review-
|
Details | Diff | Splinter Review |
Here's what it looks like in links:
Flags: (Help!) Requestee:
jkeiser: review [+] ( ________ ______________________________...
) IFrame
bryner: superreview [+] ( ________ You cannot view the attachment...
) support IFRAMEs. View the atta...
approval1.3a [ ]
mozilla-bugs: checkin [?] ( ________
)
Possible fixes: remove the parens entirely (they add minimal value) or remove
the space between them and the edit field.
|
||
Comment 2•19 years ago
|
||
Comment on attachment 208080 [details] [diff] [review]
the ()s really don't add value
If you want to go down this path, you need to remove them from all the places, in order to keep UI consistency. Like for example from the attachments table.
Attachment #208080 -
Flags: review?(justdave) → review-
|
||
Comment 3•19 years ago
|
||
It looks awful in Safari too.
Component: Attachments & Requests → User Interface
OS: Windows 98 → All
QA Contact: mattyt-bugzilla → default-qa
Hardware: PC → All
|
|
||
Comment 4•14 years ago
|
||
Fixed by bug 386600.
Assignee: timeless → guy.pyrzak
Severity: normal → trivial
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Depends on: 386600
Resolution: --- → FIXED
Whiteboard: [blocker will fix]
Target Milestone: --- → Bugzilla 4.0
|
|
||
Updated•14 years ago
|
Summary: [links] Parens around requestee field should be removed or wrapped in nobreak → Parens around requestee field should be removed
You need to log in
before you can comment on or make changes to this bug.
Description
•