Closed
Bug 560143
Opened 14 years ago
Closed 14 years ago
Page Info, Media tab allows to deselect/hide all columns
Categories
(Firefox :: Page Info Window, defect)
Firefox
Page Info Window
Tracking
()
RESOLVED
FIXED
Firefox 3.7a5
People
(Reporter: steffen.wilberg, Assigned: mozilla.bugs)
References
Details
Attachments
(1 file, 1 obsolete file)
1.16 KB,
patch
|
Details | Diff | Splinter Review |
In the Media tab of Page Info, you can hide all columns one by one using the column picker, leaving an empty tree. The user should not be able to deselect this the Address column, so it should either have primary="true" (show it in column picker as a disabled entry), or ignoreincolumnpicker="true" (don't show it in column picker at all).
Assignee | ||
Comment 2•14 years ago
|
||
I went with the primary="true" approach, otherwise it could appear that the Address option is missing.
Assignee | ||
Updated•14 years ago
|
Severity: normal → trivial
OS: Linux → All
Hardware: x86 → All
Comment 3•14 years ago
|
||
Comment on attachment 440096 [details] [diff] [review] Patch v. 1.0 >- <treecol sortSeparators="true" persist="hidden width" flex="10" >+ <treecol sortSeparators="true" primary="true" persist="hidden width" flex="10" > width="10" id="image-address" label="&mediaAddress;"/> persist="width"
Attachment #440096 -
Flags: review?(dao) → review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 5•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/00a4665760d6
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a5
You need to log in
before you can comment on or make changes to this bug.
Description
•