Olivero should use Drupal.displace() to place the mobile menu

Created on 26 November 2021, almost 4 years ago

Updated 29 May 2023, over 2 years ago

Found by @andy-blum in #3250357-16: Olivero: Elements within main content that have a high z-index will appear in front of mobile nav → , the mobile menu can be covered up by the off-canvas dialog if it comes in from the top of the viewport.

We can fix this by using the Drupal.displace() function to determine where it needs to be fixed.

🐛 Bug report

Status

Needs work

Version

11.0 🔥

Component

Olivero →

Last updated about 2 months ago

Created by

🇺🇸United States mherchel Gainesville, FL, US

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update over 2 years ago
29,397 pass
Comment over 2 years ago →
🇮🇳India gauravvvv Delhi, India
I have attached a patch for same. please review
Status changed to Needs work over 2 years ago3:27pm 29 May 2023
Comment over 2 years ago →
🇺🇸United States smustgrave
Patch doesn't match title or issue summary. So if that's the desired solution can issue summary be updated

Thanks.

Production build 0.71.5 2024