[Marble-bugs] [marble] [Bug 349453] KMZ file support for tours

[Dennis Nienhüser]

https://bugs.kde.org/show_bug.cgi?id=349453

Dennis Nienhüser <earthwings at gentoo.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |CONFIRMED
                 CC|                            |earthwings at gentoo.org
           Keywords|                            |google-code-in, junior-jobs
     Ever confirmed|0                           |1

--- Comment #1 from Dennis Nienhüser <earthwings at gentoo.org> ---
Good catch. In the simplest form it should be as easy as this patch:

diff --git a/src/lib/marble/TourWidget.cpp b/src/lib/marble/TourWidget.cpp
index 90b8c2b..0f281ce 100644
--- a/src/lib/marble/TourWidget.cpp
+++ b/src/lib/marble/TourWidget.cpp
@@ -367,7 +367,7 @@ void TourWidget::handleSliderMove( int value )
 void TourWidgetPrivate::openFile()
 {
     if ( overrideModifications() ) {
-        QString const filename = QFileDialog::getOpenFileName( q, QObject::tr(
"Open Tour" ), QDir::homePath(), QObject::tr( "KML Tours (*.kml)" ) );
+        QString const filename = QFileDialog::getOpenFileName( q, QObject::tr(
"Open Tour" ), QDir::homePath(), QObject::tr( "KML Tours (*.kml *.kmz)" ) );
         if ( !filename.isEmpty() ) {
             ParsingRunnerManager manager( m_widget->model()->pluginManager()
);
             GeoDataDocument* document = manager.openFile( filename );


However I'd rather have someone test it properly, and fix similar occurences
(e.g. importing bookmarks). For a proper support we should insert "*.kmz" as a
suggestion in the file dialog only if Marble was compiled with support for kmz
really.

-- 
You are receiving this mail because:
You are the assignee for the bug.