Closed Bug 333653 Opened 19 years ago Closed 19 years ago

[FIX]Add pref to control which filepicker is used

Categories

(Core :: Widget: Gtk, defect, P2)

x86
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

(Keywords: fixed1.8.1)

Attachments

(2 files, 1 obsolete file)

I'm tired of the GTK2 filepicker mess.
Attached patch Something like this (obsolete) — Splinter Review
Attachment #218108 - Flags: superreview?(roc)
Attachment #218108 - Flags: review?(roc)
Priority: -- → P2
Summary: Add pref to control which filepicker is used → [FIX]Add pref to control which filepicker is used
Target Milestone: --- → mozilla1.9alpha
Comment on attachment 218108 [details] [diff] [review] Something like this Do you want to add the pref to all.js?
Attachment #218108 - Flags: superreview?(roc)
Attachment #218108 - Flags: superreview+
Attachment #218108 - Flags: review?(roc)
Attachment #218108 - Flags: review+
Blocks: 326227
Attachment #218108 - Attachment is obsolete: true
Fixed.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Thanks! Would it make sense to get this for FF2 round?
Comment on attachment 218114 [details] [diff] [review] Updated to comments Yeah, probably.
Attachment #218114 - Flags: approval-branch-1.8.1?(roc)
Attachment #218114 - Flags: approval-branch-1.8.1?(roc) → approval-branch-1.8.1+
*** Bug 326227 has been marked as a duplicate of this bug. ***
Attached patch 1.8 branch patchSplinter Review
the trunk patch doesn't apply
Whiteboard: [checkin needed (1.8 branch)]
Thanks for updating this! I totally forgot about it somehow.... Checked in on 1.8 branch.
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: