Making WordPress.org

Opened 2 years ago

Closed 22 months ago

Last modified 22 months ago

#6154 closed defect (bug) (fixed)

Text not aligned properly in the mobile view

Reported by: hilayt24's profile hilayt24 Owned by:
Milestone: Priority: low
Component: Global Header/Footer Keywords:
Cc:

Description

The button text in the menu is not properly aligned in the smaller screens till 353px.
Link to the issue: https://www.awesomescreenshot.com/image/23201555?key=8558a9d0ff72e5fab0e7a5428c4f270d

Attachments (1)

23201555-3c33b0c6b15ba2fcfc770b786ee93e3f.png (51.3 KB) - added by dd32 2 years ago.

Download all attachments as: .zip

Change History (8)

#1 @dd32
2 years ago

  • Component changed from General to Global Header/Footer

I think the complaint here is that the text is on multiple lines? Is that correct @hilayt24 ?

#2 @hilayt24
2 years ago

Hi @dd3,

No, If you inspect the4 elements there is slightly left padding is there due to which the text becomes slightly right-aligned instead of the center.

Last edited 2 years ago by hilayt24 (previous) (diff)

#3 @dufresnesteven
2 years ago

@hilayt24 Did you encounter this on a specific device?

Devices below 360 pixels must have a very low usage percentage. Since the layout is still functional, albeit not perfect, I would consider not fixing this or setting its priority low.

#4 @hilayt24
2 years ago

  • Priority changed from normal to low

Hi @dufresnesteven,

You are right maybe they are low in usage percentage but still, they are available. And yes we can set its priority low for now as it is not raising serious issues.

#5 @ryelle
22 months ago

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

The spacing here has been updated since this ticket was made, and I think the issue here is resolved.

@hilayt24, if you're still seeing this bug, can you create an issue in this repo? https://github.com/WordPress/wporg-mu-plugins — that's where this project is actively being worked on.

#6 @hilayt24
22 months ago

Hi @ryelle, I can't see this issue now I think it is fixed now. Thanks for this.

#7 @hilayt24
22 months ago

  • Keywords needs-patch has-screenshots removed
Note: See TracTickets for help on using tickets.