VtEcostudies / VAL_Data_Explorers

Other
1 stars 1 forks source link

Drawer z-index is below header #3

Open MortenHofft opened 2 years ago

MortenHofft commented 2 years ago

You have a z-index of 50000 on your header The occurrence drawer has a default of z-index 1000 You can change that in the config by setting: drawerZIndex: 50001 or whatever is above your header https://github.com/VtEcostudies/VAL_GBIF_Wordpress/blob/main/gbif_data_widget.js#L19

jloomisVCE commented 2 years ago

I don't understand what problem this is fixing.

MortenHofft commented 2 years ago

Current: below header

Screenshot 2022-09-06 at 09 13 13

After fix: Above header so you can see the drawer and scientific name

Screenshot 2022-09-06 at 09 13 40
jloomisVCE commented 2 years ago

Thanks for the explanation. This looks acceptable, but do you think it could be done better by hanging the drawer from the widget, not the header?

MortenHofft commented 2 years ago

It isn't an option with the current version. I wouldn't prefer that option either I think.

If you would like me to look into that option could you then please leave an issue on https://github.com/gbif/hosted-portals/issues/new Detailing how would it behave visually/ui/ux-wise? I would need a bit of guidance I think. Thanks