Use native Mac OSX Dialog to select directory for screenshot

[henrysky]

Use native Mac OSX Dialog to select directory for screenshot
QTBUG-16722 has already been fixed (For reference: http://bugreports.qt.nokia.com/browse/QTBUG-16722)

In ConfigurationDialog.cpp, delete line435-438 and line440
I have tested it on my Mac OSX 10.9.2 and it works OK

The code:
void ConfigurationDialog::browseForScreenshotDir()
{
 QString oldScreenshorDir = StelFileMgr::getScreenshotDir();
  QString newScreenshotDir = QFileDialog::getExistingDirectory(NULL, q_("Select screenshot directory"), oldScreenshorDir, QFileDialog::ShowDirsOnly);

 if (!newScreenshotDir.isEmpty()) {
  // remove trailing slash
  if (newScreenshotDir.right(1) == "/")
   newScreenshotDir = newScreenshotDir.left(newScreenshotDir.length()-1);

  ui->screenshotDirEdit->setText(newScreenshotDir);
 }
}

[henrysky (henry-leung-2010)]

A screenshot to prove it is OK

[henrysky (henry-leung-2010)]

AddRemoveLandscapeDialog.cpp also has this problem
Start from line 116
I have tested and It also works ok for using native Mac OSX dialog

[Alexander Wolf (alexwolf)]

Can you prepare patch (bzr diff > osx.patch)?

[Alexander Wolf (alexwolf)]

A fix has been committed as revision 6691 of the trunk branch in Stellarium's Bazaar repository at Launchpad:
http://bazaar.launchpad.net/~stellarium/stellarium/trunk/revision/6691

[henrysky (henry-leung-2010)]

Sorry, I don't know how to create the patch file...
But anyway, you know what I mean and fixed it in revision 6691

[Alexander Wolf (alexwolf)]

You can use command bzr diff > osx.patch in Terminal.app, where osx.patch is file with patch.