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

[LET-1442] Disable flight mode on map interactivity controls (panning & rotation) #860

Conversation

SARodrigues
Copy link
Member

@SARodrigues SARodrigues commented Apr 3, 2024

This PR aims to disable map rotation and tilting by default.

Tracking

LET-1442

@SARodrigues SARodrigues self-assigned this Apr 3, 2024
@SARodrigues SARodrigues marked this pull request as draft April 3, 2024 11:59
@SARodrigues SARodrigues force-pushed the LET-1442-unexpected-behaviour-when-navigating-the-catalogue-page-map-with-a-trackpad branch from 6380579 to 6554a82 Compare April 3, 2024 12:02
@SARodrigues SARodrigues force-pushed the LET-1442-unexpected-behaviour-when-navigating-the-catalogue-page-map-with-a-trackpad branch from 6554a82 to ffb698e Compare April 3, 2024 12:03
@SARodrigues SARodrigues marked this pull request as ready for review April 3, 2024 12:15
Copy link
Member

@clementprdhomme clementprdhomme left a comment

Choose a reason for hiding this comment

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

There's another case to address when the map is focussed: Ctrl + Up/Down.

@SARodrigues SARodrigues marked this pull request as draft April 3, 2024 12:26
@clementprdhomme
Copy link
Member

@SARodrigues Can you review the change I submitted (if you have access to a computer that do not use MacOS)?

@clementprdhomme clementprdhomme marked this pull request as ready for review April 5, 2024 11:43
@clementprdhomme clementprdhomme merged commit b2b2250 into develop Apr 5, 2024
1 check passed
@clementprdhomme clementprdhomme deleted the LET-1442-unexpected-behaviour-when-navigating-the-catalogue-page-map-with-a-trackpad branch April 5, 2024 11:44
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