[Marble-devel] Review Request 114419: [GCI] Added support for starting KML tours from the legend

[Bernhard Beschow]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114419/#review45710
-----------------------------------------------------------


Good, we're almost there :) Some cosmetics below.


src/lib/marble/LegendWidget.h
<http://git.reviewboard.kde.org/r/114419/#comment32650>

    not needed, please remove



src/lib/marble/LegendWidget.h
<http://git.reviewboard.kde.org/r/114419/#comment32651>

    Please change to "const QString &" and give the parameter a name.



src/lib/marble/LegendWidget.cpp
<http://git.reviewboard.kde.org/r/114419/#comment32652>

    not needed, please remove



src/lib/marble/LegendWidget.cpp
<http://git.reviewboard.kde.org/r/114419/#comment32653>

    please connect to LegendWidget::tourLinkClicked() directly (using SIGNAL(tourClicked(...)) and remove handleTourLinkClick()



src/lib/marble/MarbleLegendBrowser.h
<http://git.reviewboard.kde.org/r/114419/#comment32654>

    same as above, please use "const QString &" and name the parameter


- Bernhard Beschow


On Dec. 14, 2013, 12:46 a.m., Mihail Ivchenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114419/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2013, 12:46 a.m.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Added support for starting KML tours from the legend
> GCI task: http://www.google-melange.com/gci/task/view/google/gci2013/5796158626594816
> 
> Example of using:
> Put this into <legend></legend> element of .dgml:
>       <section name="tours" checkable="false" spacing="12">
> 	  <heading>Tours:</heading> 
> 	  <item name="geographic-pole">
> 	      <text><![CDATA[<a href="tour://planet/map/test_tour.kml">Test Tour</a>]]></text>
> 	  </item>
>       </section>
> 
> 
> Diffs
> -----
> 
>   src/apps/marble-ui/ControlView.h 322708b 
>   src/apps/marble-ui/ControlView.cpp fc9022f 
>   src/lib/marble/LegendWidget.h e56966a 
>   src/lib/marble/LegendWidget.cpp 280fd64 
>   src/lib/marble/MarbleLegendBrowser.h e7fb4b5 
>   src/lib/marble/MarbleLegendBrowser.cpp 644dd0f 
>   src/lib/marble/TourWidget.h c18c849 
>   src/lib/marble/TourWidget.cpp d595418 
> 
> Diff: http://git.reviewboard.kde.org/r/114419/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mihail Ivchenko
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20131215/8e13e789/attachment.html>