Making WordPress.org

Opened 20 months ago

Closed 20 months ago

Last modified 20 months ago

#6096 closed defect (bug) (fixed)

In meta module pages the hamburger menu is not proper.

Reported by: smit08's profile smit08 Owned by: ryelle's profile ryelle
Milestone: Priority: normal
Component: Handbooks Keywords: has-patch needs-testing
Cc:

Description

In meta module pages like meta handbook, meta trac, meta projects etc. The hamburger menu is not proper. It's in fixed position so floating all over the page. And due to that not able to view all the hamburger menu items.

Attachments (3)

Screenshot 2022-02-11 at 3.53.54 PM.png (719.2 KB) - added by smit08 20 months ago.
6096.patch (948 bytes) - added by hilayt24 20 months ago.
The burger menu is now visible in the smaller screens previously it goes behind the main header.
#6096.patch (868 bytes) - added by kajalgohel 20 months ago.
Hamburger menu start visible at 876px and below. I have added the css for 876px and below.

Download all attachments as: .zip

Change History (9)

@hilayt24
20 months ago

The burger menu is now visible in the smaller screens previously it goes behind the main header.

#1 @hilayt24
20 months ago

  • Keywords has-patch added; needs-patch removed

@kajalgohel
20 months ago

Hamburger menu start visible at 876px and below. I have added the css for 876px and below.

#2 @kajalgohel
20 months ago

  • Keywords needs-testing added

#3 @dd32
20 months ago

  • Component changed from General to Handbooks
  • Milestone Plugin Directory v3 - Future deleted

Page in question: https://make.wordpress.org/meta/handbook/

Instead of using px offsets though, we should use the headers CSS variables it offers to detect the offset (It updates them with the various header sizes)

#4 @ryelle
20 months ago

  • Owner set to ryelle
  • Status changed from new to assigned

#5 @ryelle
20 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 11573:

Make: Set mobile menu on top of the header.

Props smit08, kajalgohel, dd32.
Fixes #6096.

#6 @ryelle
20 months ago

In 11574:

Developer: Set mobile menu on top of the header.

Props kajalgohel.
See #6096.

Note: See TracTickets for help on using tickets.