Making WordPress.org

Opened 8 months ago

Closed 8 months ago

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

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

#1 @hilayt24
8 months ago

  • Keywords has-patch added; needs-patch removed

@kajalgohel
8 months ago

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

#2 @kajalgohel
8 months ago

  • Keywords needs-testing added

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

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

#5 @ryelle
8 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
8 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.