Problem/Motivation

If you have the navigation module installed and use it in combination with gin (gin toolbar is also installed), on the desktop narrowing down the browser window looks like that (screenshot is in safari but same result in for example edge on macos)

the node edit form on a deskop narrowed down to mobile viewport with cramped top bar components overlapping

and on the iphone it looks the same

the node edit form on an iphone display with cramped top bar components overlapping

Steps to reproduce

simply go to /node/1/edit

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#7 After Patch.png94.84 KBsandeep_k
#7 Before Patch.png134.3 KBsandeep_k
topbar_mobile.PNG668.52 KBrkoller
topbar_desktop.jpg42.42 KBrkoller

Issue fork gin-3543540

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

rkoller created an issue. See original summary.

rkoller’s picture

rkoller
he/him
Primary language German
Location Nürnberg, Germany
commented
Issue summary: View changes
rkoller’s picture

rkoller
he/him
Primary language German
Location Nürnberg, Germany
commented
Title: with the navigation module installed many elements in the top bar are overlapping in mobile viewport » with the navigation module installed many elements in the top bar are overlapping on the mobile viewport
anirudhsingh19’s picture

anirudhsingh19 commented
Assigned: Unassigned » anirudhsingh19

working on this.

anirudhsingh19 opened merge request !690

anirudhsingh19’s picture

anirudhsingh19 commented
Assigned: anirudhsingh19 » Unassigned
Status: Active » Needs review

Please review.

sandeep_k’s picture

sandeep_k commented
StatusFileSize
new Before Patch.png134.3 KB
new After Patch.png94.84 KB

I have tested this on Drupal Version 11.2.0 and am able to reproduce this issue. "MR !690 mergeable" was applied successfully, and the issue is fixed now.
Added before and after results here, keeping in review for more reviews.

ahsannazir’s picture

ahsannazir commented

The MR fixes the issue by overriding the grid with simple flex properties. The changes seem to be working fine as expected.
Hence moving to RTBC

ahsannazir’s picture

ahsannazir commented
Status: Needs review » Reviewed & tested by the community

jurgenhaas made their first commit to this issue’s fork.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Reviewed & tested by the community » Needs review

There was a merge conflict that I resolved, but I'm uncertain if everything is still correct, so please review once more.