Option 'Use desktop environment native dialogs' broken when saving sessions?

Submitted by schnitzeltony on
Forums
qtractor

Hi,

first I thought it was due to debugging with qtcreator but then I saw that release build for Raspi behaves same: When trying to save a session (Save As) or selecting a path for a fresh session qtractor hangs. I disabled the option mentioned in thread-subject to use Qt dialogs and that worked around the problem (although Qt dialogs are not good at removable devices)

FWIW:
* On my PC (fedora / debug with qtcreator) Qt 5.7.1 is installed / Qtractor git HEAD
* On Raspi Qt version 5.9.1+ (sources are taken form git) is installed / QTractor 0.8.4

Have no idea when this bug came in..

One final question: Is this the correct forum to discuss these kind of issues or do you prefer me to open a github issue

Subscribe to this forum topic

Submitted by rncbc on Mon, 9 Oct 2017 - 05:32

Permalink

yes, that seems to be the case the DE native dialogs are prone to lockup/freezing the whole event-loop, most specially the ones that are based on gtk3 as from gnome and/or xfce.

it is the main reason why that option is set off by default since v0.8.1; it has been pointed out here several times before though.

byee

Submitted by rncbc on Mon, 9 Oct 2017 - 21:45

Permalink

re. Would you accept a patch changing parent members to NULL at all locations?

sure.

but please, be warned of some probable and nasty UX side-effects then eg. you having a file-dialog dang open and then accidentally somewhere the desktop decides to popup something else and you can't find where you really are or were because it's been covered up by some other window and qtractor et al. all seems to non-responding to any user input, no mather you try clicking here and there on any of its main windows--just not hitting the right one, the dangling file-open/save dialog that is--been there before i say, not a pretty experience either ;)

byee

Submitted by schnitzeltony on Wed, 11 Oct 2017 - 07:33

Permalink

The more I think of this issue I start disliking my solution: Setting parent to NULL adds an extra entry in taskbar for file-dialogs. OK that is better than hanging but...

Anyway - I will send out the pull request and leave the decision upon you.

Submitted by schnitzeltony on Wed, 11 Oct 2017 - 23:33

Permalink

* your rework is far better than my solution. Just saw that you removed changes for VST. Have no plugins to test but I am sure there are reasons for it (think you spent many more nights on qtractor than I ever will..)
* did the same tests I did when writing the patch it looks same -> working -> party time

Thanks for taking care

Add new comment

The content of this field is kept private and will not be shown publicly.
About text formats

Markdown

  • Parses markdown and converts it to HTML.
  • Allowed HTML tags: <a href hreflang> <em> <strong> <cite> <blockquote cite> <code> <ul type> <ol start type='1 A I'> <li> <dl> <dt> <dd> <h2 id='jump-*'> <h3 id> <h4 id> <h5 id> <h6 id> <img src alt height width> <strike> <pre> <p> <br>
  • Lines and paragraphs break automatically.

Filtered HTML

  • Allowed HTML tags: <a href hreflang> <em> <strong> <cite> <code> <ul type> <ol start type> <li> <dl> <dt> <dd> <b> <i> <pre> <img src alt height width> <strike>
  • Lines and paragraphs break automatically.
  • Web page addresses and email addresses turn into links automatically.

Plain text

  • No HTML tags allowed.
  • Lines and paragraphs break automatically.
  • Web page addresses and email addresses turn into links automatically.
File attachments
Unlimited number of files can be uploaded to this field.
2 MB limit.
Allowed types: jpg jpeg gif png txt doc docx xls xlsx pdf ppt pps odt ods odp zip gz bz2 xz patch diff wav ogg flac ogv mp4 qtz.
CAPTCHA
Image CAPTCHA
Get new captcha!
Enter the characters shown in the image.