Map widget reversed zoom

FlightGear Flight Simulator: free open-source multiplatform flight sim

Brought to you by: curt, jmturner, stuartbuchanan, torstendreyer

#485 Map widget reversed zoom

Milestone: 2.6.0

Status: Fixed

Owner: nobody

Labels: GUI (96) fixable (42)

Priority: Medium

Updated: 2011-11-09

Created: 2011-11-08

Creator: Anonymous

Private: No

Originally created by: gijsrooy
Originally owned by: gijsrooy

What steps will reproduce the problem?
1. Open the Equipment > Map dialog.
2. Click the zoom buttons (+ and -).

What is the expected output? What do you see instead?
+ should zoom in, while - zooms out. At least that's what it is on most applications. In FlightGear - zooms in, while + zooms out.

What FlightGear version (or GIT date)?
Today's Git and fgdata.

What operating system and graphics card?
Confirmed on Windows and Linux.

Discussion

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-08

Originally posted by: zakalawe@mac.com

Discussed on IRC, will expose the max zoom as a property, so the GUI can display the values either way around.

*Originally posted by:* [zakalawe@mac.com](http://code.google.com/u/zakalawe@mac.com/) Discussed on IRC, will expose the max zoom as a property, so the GUI can display the values either way around.

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-08

Originally posted by: gijsrooy

(No comment was entered for this change.)

Status: Dev

*Originally posted by:* [gijsrooy](http://code.google.com/u/gijsrooy/) *\(No comment was entered for this change.\)* **Status:** Dev

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-08

Originally posted by: bre... (code.google.com)@gmail.com

+1. Had noticed the same. Would be nice to invert the zoom buttons.

*Originally posted by:* [bre...@gmail.com](http://code.google.com/u/112567806978334989774/) +1. Had noticed the same. Would be nice to invert the zoom buttons.

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-08

Originally posted by: zakalawe@mac.com

Working on it - due to some PUI limitations, it's proving easier to flip the zoom in the C++ side.

*Originally posted by:* [zakalawe@mac.com](http://code.google.com/u/zakalawe@mac.com/) Working on it - due to some PUI limitations, it's proving easier to flip the zoom in the C++ side.

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-08

Originally posted by: zakalawe@mac.com

Right, zoom sense is flipped at the C++ level. I've also exposed the max-zoom as a property, but unfortunately, property-adjust commands don't support reading their limits from properties, only constants, so the value is still duplicated in the dialog XML.

https://gitorious.org/fg/flightgear/commit/d2d3fc054e942139b124053d680f3d4a0a63a77f

Status: Fixed

*Originally posted by:* [zakalawe@mac.com](http://code.google.com/u/zakalawe@mac.com/) Right, zoom sense is flipped at the C++ level. I've also exposed the max-zoom as a property, but unfortunately, property-adjust commands don't support reading their limits from properties, only constants, so the value is still duplicated in the dialog XML. [https://gitorious.org/fg/flightgear/commit/d2d3fc054e942139b124053d680f3d4a0a63a77f](https://gitorious.org/fg/flightgear/commit/d2d3fc054e942139b124053d680f3d4a0a63a77f) **Status:** Fixed

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "codetickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2011-11-09

Originally posted by: gijsrooy

Thanks James!

Labels: Milestone-2.6.0

*Originally posted by:* [gijsrooy](http://code.google.com/u/gijsrooy/) Thanks James\! **Labels:** Milestone-2.6.0

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.