Making WordPress.org

Opened 3 years ago

Closed 3 years ago

Last modified 3 years 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 3 years ago.
6096.patch (948 bytes) - added by hilayt24 3 years 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 3 years 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
3 years ago

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

#1 @hilayt24
3 years ago

  • Keywords has-patch added; needs-patch removed

@kajalgohel
3 years ago

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

#2 @kajalgohel
3 years ago

  • Keywords needs-testing added

#3 @dd32
3 years 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
3 years ago

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

#5 @ryelle
3 years 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
3 years ago

In 11574:

Developer: Set mobile menu on top of the header.

Props kajalgohel.
See #6096.

Note: See TracTickets for help on using tickets.