Skip to content
New issue

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

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

Already on GitHub? # to your account

[PLAY-881] Fixing the map kit button height for Safari #2655

Merged
merged 2 commits into from
Jul 21, 2023

Conversation

carloslimasd
Copy link
Contributor

@carloslimasd carloslimasd commented Jul 19, 2023

What does this PR do?
Fix the Map kit's nav buttons that are oversized and breaking their own layout in Safari.

How to test? Steps to confirm the desired behavior:

  1. Go to Map kit in Safari
  2. Check the map buttons

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@carloslimasd carloslimasd added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Jul 19, 2023
@carloslimasd carloslimasd self-assigned this Jul 19, 2023
@carloslimasd carloslimasd requested a review from a team as a code owner July 19, 2023 18:49
@app-milano app-milano bot temporarily deployed to pr2655 July 19, 2023 19:11 Inactive
@nidaqg nidaqg added the Ready for Release merged to master, ready for a versioned released label Jul 19, 2023
@nidaqg nidaqg added this pull request to the merge queue Jul 21, 2023
Merged via the queue into master with commit 25e5302 Jul 21, 2023
@nidaqg nidaqg deleted the PLAY-881-map-kit-button-safari branch July 21, 2023 18:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants