Skip to content

fix: improves display of map status and fit to feature #657

Merged
latin-panda merged 9 commits intomainfrom
ui-fixes-for-map
Feb 18, 2026
Merged

fix: improves display of map status and fit to feature #657
latin-panda merged 9 commits intomainfrom
ui-fixes-for-map

Conversation

@latin-panda
Copy link
Collaborator

@latin-panda latin-panda commented Feb 17, 2026

I have verified this PR works in these browsers (latest versions):

  • Chrome
  • Firefox
  • Safari (macOS)
  • Safari (iOS)
  • Chrome for Android
  • Not applicable

What else has been done to verify that this works as intended?

Manual testing and e2e tests passing

Why is this the best possible solution? Were any other approaches considered?

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Do we need any specific form for testing your changes? If so, please attach one.

What's changed

  • The status message Capturing location... was wrapping instead of using the available space when loading on mobile devices.
  • The advanced panel wasn't closing when exiting full-screen mode. This is particularly inconvenient on mobile devices, where we hide the advance button.
  • The "fit view to feature" was using the current zoom. If the zoom level is 1 but the feature is too large, it doesn't fit within the map viewport. It now calculates the feature's zoom level.

@changeset-bot
Copy link

changeset-bot bot commented Feb 17, 2026

🦋 Changeset detected

Latest commit: f72b34e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@getodk/web-forms Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@latin-panda
Copy link
Collaborator Author

Now that all enhancements have been implemented, I ran regression tests and found 3 small issues, which I've fixed.
@garethbowen could you please review when you have time?

Copy link
Collaborator

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

One code suggestion inline. Approved to unblock.

@latin-panda latin-panda merged commit da6dc94 into main Feb 18, 2026
97 of 98 checks passed
@latin-panda latin-panda deleted the ui-fixes-for-map branch February 18, 2026 10:11
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

Comments