Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adopt term 'Map reference point' #2029

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dl3sdo
Copy link
Member

@dl3sdo dl3sdo commented Jan 20, 2022

In the 'Rotate map', 'Change map scale' and 'Change scale factor' dialogs, the rotation/scaling center can be chosen to be the 'Georeferencing reference point', even for local CRS.
This is confusing since the reference point is based on map coordinates (e.g., in mm) and not on geographic coordinates.
In the 'Options' sections below the term 'Adjust georeferencing reference point' also only refers to the
coordinates of the map reference point.
This change implements Scott Turner's (@pkturner) proposal to use 'Map reference point' and 'Adjust coordinates of map reference point' instead.
It supersedes #2014.

@dg0yt dg0yt changed the title ScaleRotateDialogs: Change notion 'georeferencing reference point' ScaleMapDialog, RotateMapDialog: Adopt term 'Map reference point' Jan 23, 2022
@dl3sdo dl3sdo force-pushed the map-scale-rotate branch from ebb3724 to 07eef0e Compare March 24, 2022 17:49
@dl3sdo
Copy link
Member Author

dl3sdo commented Mar 24, 2022

Rebase and split of previous commit to separate changes for each affected file.

@dl3sdo dl3sdo changed the title ScaleMapDialog, RotateMapDialog: Adopt term 'Map reference point' Adopt term 'Map reference point' Dec 29, 2024
In the 'Rotate map', 'Change map scale' and 'Change scale factor'
dialogs, the rotation/scaling center can be chosen to be
the 'Georeferencing reference point', even for local CRS.
This is confusing since the reference point is based on map coordinates
(e.g., in mm) and not on geographic coordinates.
In the 'Options' section below the term 'Adjust georeferencing
reference point' also only refers to the coordinates of the map
reference point.
@dl3sdo
Copy link
Member Author

dl3sdo commented Dec 29, 2024

@pkturner: The initial commit ignored the StretchMapDialog dialog that uses the same wordings. Please check, thank you.

@dl3sdo dl3sdo requested a review from pkturner December 29, 2024 19:50
Copy link
Contributor

@pkturner pkturner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change in terminology from "georeferencing reference point" to "map reference point" is good all around.

I added related (though independent) issue #2310.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants