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)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Firefox 3.7a5

People

(Reporter: steffen.wilberg, Assigned: mozilla.bugs)

References

Details

Attachments

(1 file, 1 obsolete file)

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).
The equivalent SeaMonkey bug is Bug 168908.
Blocks: 168908
Attached patch Patch v. 1.0 (obsolete) — Splinter Review
I went with the primary="true" approach, otherwise it could appear that the Address option is missing.
Assignee: nobody → mozilla.bugs
Status: NEW → ASSIGNED
Attachment #440096 - Flags: review?(dao)
Severity: normal → trivial
OS: Linux → All
Hardware: x86 → All
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+
Attached patch Patch v 1.1Splinter Review
Updated.
Attachment #440096 - Attachment is obsolete: true
Keywords: checkin-needed
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.

Attachment

General

Created:
Updated:
Size: