Bug 249628

Summary: MarbleWidget::zoomView(MarbleWidget::zoom()) is off by one
Product: [Applications] marble Reporter: Volker Lanz <vl>
Component: generalAssignee: marble-bugs
Status: RESOLVED FIXED    
Severity: normal CC: bastianholst
Priority: NOR    
Version: unspecified   
Target Milestone: 1.0 (KDE 4.6)   
Platform: Compiled Sources   
OS: Microsoft Windows   
Latest Commit: Version Fixed In:

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