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

Fix pointer events on map controls #672

Merged
merged 3 commits into from
Sep 25, 2023
Merged

Fix pointer events on map controls #672

merged 3 commits into from
Sep 25, 2023

Conversation

danielfdsilva
Copy link
Collaborator

The fact that this element takes up the full width was causing interaction problems. If you tried to draw on top of it, it was not possible.
image

This PR removes pointer events from the parents, including them only on the children.

@nerik This may need to be applied to the new map approach as well. I'd ask that you please check if it is needed.

@netlify
Copy link

netlify bot commented Sep 23, 2023

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 5276d17
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/650eca2553e20100082237bc
😎 Deploy Preview https://deploy-preview-672--veda-ui.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@nerik nerik left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@nerik nerik merged commit 77f3946 into main Sep 25, 2023
7 checks passed
@nerik nerik deleted the fix/pointer-events branch September 25, 2023 09:38
@nerik nerik mentioned this pull request Sep 28, 2023
nerik added a commit that referenced this pull request Oct 2, 2023
Fixed the issue addressed
[here](#672) on the new map
component.
hanbyul-here added a commit that referenced this pull request Oct 12, 2023
## This PR, merged to `ghg`

### Features

- Analysis define
  - New step-by-step layout and various UI tweaks
  - Added sticky footer, moved submit and cancel buttons there
  - Added an AoI preset for North America
- Datasets are not filtered from `/collections` metadata (spatial,
temporal extent)
  - Dates are now set by default to 2018 - 2022
  - Replaced date presets with "2018 - 2022" and "last 10 years"
  - Date selection not possible anymore before 01-01-1980
- Prevent users from selecting datasets when time range selected would
mean fetching too many STAC items
- Analysis results page: set a hardcoded limit on the number of items
that can be requested as a failsafe mechanism for ☝️

### Bugfixes

- Removed logs in production
(US-GHG-Center/veda-config-ghg#152)
- Temporary fix of critical error encoutered during demos
(#680)
- Remove Shadow Scroll as a preventative method for
(#691)
- Check only day level equity to validate dates
(#676)
- Remove unneded attributes
(#694)
- BlockMap initial position error
(#687)

### Copy changes
- [Updated define
copy](f1e7171)
- [Disclaimer on results
page](54cf01e)


--- 

## Other PRs, merged to `main`

### Added in veda UI but made configurable (so not affecting
veda-config):
- #673
- #665
- #664

### Bugfixes
- #672
- #670 /
#666
- #662
- #691
- #650
- #676
- #694
- #687
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