nsFilePicker is broken in qt desktop

RESOLVED FIXED

Status

defect
RESOLVED FIXED
9 years ago
3 years ago

People

(Reporter: tatiana, Assigned: tatiana)

Tracking

Trunk
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Assignee

Description

9 years ago
Open qt desktop Firefox build

File -> Open File

EXPECTED:
file dialog

ACTUAL:
broken_file_dialog.png
Assignee

Comment 1

9 years ago
Posted patch patch (obsolete) — Splinter Review
I don't see any reason to embed QFileDialog with QGraphicsProxyWidget creature
Assignee: nobody → tanya.meshkova
Status: NEW → ASSIGNED
Attachment #504698 - Flags: review?(romaxa)
Comment on attachment 504698 [details] [diff] [review]
patch


>+    MozQWidget *parentMozWidget = static_cast<MozQWidget*>(parent->GetNativeData(NS_NATIVE_WIDGET));
GetNativeData may return null...
check for null, add NS_WARNING and set parentWidget = 0 if parentMozWidget == null
>+    QWidget *parentWidget = parentMozWidget->getReceiver()->GetViewWidget();
>+
>-    }
>+    mDialog = new QFileDialog(parentWidget, QString::fromUtf16(str.get()));
>
Attachment #504698 - Flags: review?(romaxa) → review-
Assignee

Comment 3

9 years ago
Posted patch patch v2Splinter Review
Attachment #504698 - Attachment is obsolete: true
Attachment #504709 - Flags: review?(romaxa)
Comment on attachment 504709 [details] [diff] [review]
patch v2

By checking different implementations, I noticed that all other ports are creating Dialogs in ::Show function and in ::InitNative function it is remembering parent, title...

Don't know exactly why it is done this way, but probably it make sense for Qt port do similar implementation...
Attachment #504709 - Flags: review?(romaxa) → review+
Assignee

Updated

9 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/3a9a6b749df6
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.