- Issue created by @ckrina
- Assigned to kostyashupenko
- Merge request !143The footer shadow disappears when a button is hovered → (Open) created by kostyashupenko
- 🇷🇺Russia kostyashupenko Omsk
It was a bit tricky, but working now and working even better than before from my pov.
Previous shadows were added using CSS gradients technique. However we can't use this method, since inner elements in the tree contains background colors (on hover, active, etc states) which are overlapping parent gradient (that's why shadow disappeared @ckrina on hovering).
Now shadows are controlled by javascript (which is OK i think, and there will be no shadows at all if js is disabled in browser).
Mobile behavior:
Desktop behavior:
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 7:41am 30 November 2023 - Status changed to RTBC
about 1 year ago 10:20am 11 December 2023 - 🇮🇳India pragati_kanade Pune
I have reviewed #6 and it is working fine as per the proposed solution so I am moving it to RTBC.
Thanks. - Status changed to Needs work
about 1 year ago 10:28am 11 December 2023 - 🇷🇸Serbia finnsky
Sorry, i didn't change status after MR discussion.
I still think that we can move it outside of sidebar. - Assigned to finnsky
- Merge request !148The footer shadow disappears when a button is hovered - #3405006 → (Open) created by finnsky
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 12:17pm 11 December 2023 - 🇷🇸Serbia finnsky
Rerolled previously added script here. It seems me more delicate approach. Please review.
- Status changed to RTBC
about 1 year ago 6:21am 13 December 2023 - 🇮🇳India pragati_kanade Pune
Hi @finnsky I have review this MR and It looks good to me. Footer shadow is also not getting disappear after hover on buttons.
Hence changing status to "RTBC".
Thanks. - Status changed to Needs work
about 1 year ago 8:20am 13 December 2023 - 🇷🇸Serbia finnsky
Sorry, i retested my work. Seems i forgot to manage header shadow in expanded state. Sending back to NW.
- 🇮🇳India prashant.c Dharamshala
@finnsky
I am not sure, this issue is related to this or not. On hover of last menu item, the bottom border disappears/overflows.
Thanks! - Assigned to chetansonawane
- 🇮🇳India chetansonawane Gujarat
chetansonawane → changed the visibility of the branch single-overflow-script-3405006 to hidden.
- 🇮🇳India chetansonawane Gujarat
chetansonawane → changed the visibility of the branch single-overflow-script-3405006 to active.
- Issue was unassigned.
- Status changed to RTBC
8 months ago 10:18am 16 April 2024 - 🇮🇳India chetansonawane Gujarat
@finnsky,
I think the whole UI behavior is changed now, I can not see the dropdown functionality while hovering on the menu item, on hover of the first level menu it expand left side and on hover of second level menu there is dropdown, but UI is different, so it is working fine, we can close this issue.
- Status changed to Needs work
8 months ago 10:56am 16 April 2024 - 🇷🇸Serbia finnsky
@chetansonawane thank you for testing.
I think it should be rebased or even recreated first.
Problem still present in current 1.x - 🇷🇸Serbia finnsky
Here also we need to fix this one.
I suggest to add to resize observer for that shadows.
- First commit to issue fork.
- 🇮🇳India gauravvvv Delhi, India
Gauravvvv → changed the visibility of the branch single-overflow-script-3405006 to hidden.
- 🇮🇳India gauravvvv Delhi, India
Gauravvvv → changed the visibility of the branch 3405006-the-footer-shadow to hidden.
- Status changed to Needs review
7 months ago 4:00am 17 May 2024 - Status changed to Needs work
7 months ago 7:34am 17 May 2024 - 🇷🇸Serbia finnsky
Thank you for backporting this.
Some notes.
Since that original script was written lot of things were changed.
We need to rework at and adapt to current logic.1. I see shadow on mobile but not on desktop
2. I see broken header and wrong padding relative to footer.
https://gyazo.com/f08ad11ee5f5d21f4290af428d220d723. I see some flickering on popover open
https://gyazo.com/b80215700f117abb7350000800758d08
probably not related to this fix but better to check - First commit to issue fork.
- 🇮🇳India ahsannazir
The shadow is shown both on Desktop & Mobile when
data-detected-y-overflow="true"
The 2nd point mentioned in #32 about flickering needs to be investigated more and fixed accordingly.
- Status changed to Needs review
4 months ago 6:48am 12 August 2024 - 🇮🇳India Kanchan Bhogade
Hi
I've tested MR 8106 on Drupal 11
The MR is applied cleanly...Checked for the #32 1 and 2 point
1. Added shadow on desktop
2. Added shadow on Mobile for headerAttaching Screenshots
- Status changed to Needs work
4 months ago 1:56pm 12 August 2024 - 🇺🇸United States smustgrave
Issue summary is not complete, proposed solution is empty and User Interface section is empty and this appears to be a UI fix.
Have not reviewed
- 🇮🇳India ahsannazir
@gauravvvv Can you please add/update the issue summary with proposed solution