Bug 249628

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

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