← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~phill-ridout/openlp/issue1000 into lp:openlp

 

> Why do we need normpath.  No other path inputs use this.

For some reason QFileDialog.getOpenFileName returns the path with
forward slashes instead on back slashes on windows. Where as
QFileDialog.getExistingDirectory returns it correctly.

I have just noticed we have the same issue on the advanced tab under
the default image. I'm guessing it doesn't matter too much (as Opening
and Saving works) but it looks just wrong where the user can see it!


> Validating the save on exit is not a good user experience.  It should be done on moving to the next tab after selecting Image.  You have to return 5 tabs and have generated a preview and requested save before this is triggered.

Good point

-- 
https://code.launchpad.net/~phill-ridout/openlp/issue1000/+merge/121249
Your team OpenLP Core is subscribed to branch lp:openlp.


References