Bug 249628 - MarbleWidget::zoomView(MarbleWidget::zoom()) is off by one
Summary: MarbleWidget::zoomView(MarbleWidget::zoom()) is off by one
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Microsoft Windows
: NOR normal
Target Milestone: 1.0 (KDE 4.6)
Assignee: marble-bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-31 15:11 UTC by Volker Lanz
Modified: 2010-10-13 23:04 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Volker Lanz 2010-08-31 15:11:06 UTC 
Version:           unspecified (using Devel) 
OS:                MS Windows

This little snippet:

-----

Marble::MarbleWidget* marbleWidget = new Marble::MarbleWidget(this);

// ...

const int zoom = marbleWidget->zoom();

// ...

marbleWidget->zoomView(zoom);

-----

should not, AFAICS, change the zoom for a marble widget; it zooms out a tiny bit, however. Calling marbleWidget->zoomView(zoom + 1) fixes this, so I suspect there's an off by one error somewhere.


Reproducible: Always
Comment 1 Dennis Nienhüser 2010-10-13 20:44:42 UTC 
SVN commit 1185567 by nienhueser:

Round, don't cut.
BUG: 249628
CCBUG: 254000

 M  +1 -1      MarbleMap.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1185567
Comment 2 Bastian Senst 2010-10-13 23:04:20 UTC 
SVN commit 1185602 by bholst:

* Round, don't cut.
* Make sure the right zoom-level is saved.

BUG: 249628

backport r1185567
backport r1185591

 M  +10 -3     MarbleMap.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1185602