Bug 202072

Summary:	MGI::positions() isn't const, positivePosition() reinterpretation
Product:	[Applications] marble	Reporter:	Torsten Rahn <rahn>
Component:	general	Assignee:	marble-bugs
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	jensmh
Priority:	NOR
Version:	unspecified
Target Milestone:	0.8 (KDE 4.3)
Platform:	Compiled Sources
OS:	Unspecified
Latest Commit:		Version Fixed In:

Description Torsten Rahn 2009-07-31 13:02:17 UTC

Version:            (using Devel)
Installed from:    Compiled sources

Currently the positions() method isn't a const getter. 

We might want to change this.

In related matters the ScreenGraphicsItem::positions method already reinterpretes the negative position as a positive one (which I feel should maybe better be done in AbstractFloatItem itself, so that it's still possible to create ScreenGraphicsItems that are partially out of the viewport ).

Also as another suggestion by ariya we might consider using alignment properties instead of the current implementation. However this might introduce additional complexity, so this should be discussed.

Comment 1 jensmh 2009-08-01 11:17:54 UTC

Fixed in svn revision 1005225.