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.

Sign up for GitHub

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

Jump to bottom

Fix navigation menu being covered by map menu options #51

Closed
mpitblado opened this issue Dec 4, 2023 · 3 comments · Fixed by #53 or #54
Closed

Fix navigation menu being covered by map menu options #51

mpitblado opened this issue Dec 4, 2023 · 3 comments · Fixed by #53 or #54
Assignees
@mpitblado
Labels
bug Something isn't working
Milestone
1.0.2

Comments

@mpitblado
Copy link
Collaborator

Clicking on the navigation menu while in the map view leads to the navigation menu being covered up by the map menu (so the user cannot access the Specimens tab). This is likely a quick z-index fix.

image
@mpitblado mpitblado added the bug Something isn't working label Dec 4, 2023
@mpitblado mpitblado added this to the 1.0.2 milestone Dec 4, 2023
@mpitblado mpitblado self-assigned this Dec 4, 2023
@mpitblado mpitblado linked a pull request Dec 5, 2023 that will close this issue
Push to staging to check z-index effect on page #53
Merged
MortenHofft added a commit to gbif/gbif-web that referenced this issue Dec 5, 2023
@MortenHofft
fix: close gbif/hp-beaty-biodiversity-museum#51
e68cfd5
@MortenHofft
Copy link
Member

MortenHofft commented Dec 5, 2023 •
edited

This could be changed 3 places. in the react widgets, the theme or in your site.
It made best sense to do it in the widgets i found. I'll get it into production later today i hope.

Changing it in your site would leave it as an issue for everyone else

@MortenHofft
Copy link
Member

MortenHofft commented Dec 5, 2023 •
edited

Ahh . i only just now realise that it is because we did custom styling of your menu dropdown. Anyway, above should fix it either way. It didn't need to be that high on the map widget

@mpitblado
Copy link
Collaborator Author

Can confirm now fixed due to the push to the widget! Thanks

@mpitblado mpitblado linked a pull request Dec 5, 2023 that will close this issue
Put z-index to 100 for nav menu to go over everything #54
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mpitblado mpitblado

Labels
bug Something isn't working
Projects
None yet
Milestone
1.0.2
2 participants
@MortenHofft @mpitblado