[Marble-bugs] [marble] [Bug 318057] New: Add a MapInfoDialog::popup() method that pans the map appropriately

[Dennis Nienhüser]

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

            Bug ID: 318057
           Summary: Add a MapInfoDialog::popup() method that pans the map
                    appropriately
    Classification: Unclassified
           Product: marble
           Version: 1.6.0 (KDE 4.11.0)
          Platform: Other
                OS: Linux
            Status: UNCONFIRMED
          Keywords: google-code-in, junior-jobs
          Severity: task
          Priority: NOR
         Component: general
          Assignee: marble-bugs at kde.org
          Reporter: earthwings at gentoo.org

Currently the MapInfoDialog is shown via a setVisible( true ) call. It pops up
at the current geo position in screen coordinates, which is fine.
It should also be possible however to have the dialog open after it recenters
the map such that its content becomes ideally visible. The geo position
associated with the map dialog should move somewhere near the window center
then (shifted a bit left/right/above/below depending on the alignment).

A possible implementation could go like that:
- add a method setMarbleWidget( MarbleWidget* widget ) to MapInfoDialog. Have
it store the passed pointer in a new member variable
- Change MarbleWidget.cpp to call that method where it sets up m_mapInfoDialog
- add a method popup() to MapInfoDialog
- Inside, store a bool m_adjustMap=true and call setVisible( true )
- Adjust render() to check for m_adjustMap. If set, recenter (using
MarbleWidget*) appropriately. Bonus points for shifting it a bit from the
center based on the alignment

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