Making WordPress.org

Opened 3 years ago

Closed 2 days ago

#5680 closed defect (bug) (fixed)

Support Theme: Improve single topic mobile view (take 2)

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: coffee2code's profile coffee2code
Milestone: Priority: low
Component: Support Forums Keywords: has-screenshots has-patch
Cc:

Description

Previously: #4203

Single topic mobile view (at 480px or less) has a couple issues:

  • The "also viewing" banner has some extra padding at the top.
  • The "Plugin Author" badge also has some extra padding on the left and is disconnected from the line it's supposed to be connected to. Might not be a big deal, but when compared to the desktop view, it gives a feeling that something does not look quite right.

See the screenshot for both issues.

Attachments (3)

5680.png (25.7 KB) - added by SergeyBiryukov 3 years ago.
5680.1.patch (1.5 KB) - added by vladytimy 3 years ago.
Removes space between line and badge on mobile devices
5680.1_after.jpg (122.1 KB) - added by vladytimy 3 years ago.

Download all attachments as: .zip

Change History (9)

@SergeyBiryukov
3 years ago

#1 @vladytimy
3 years ago

[related] #2552 has 2552.1.patch that fixes the first issue addressed in this ticket.

Last edited 3 years ago by SergeyBiryukov (previous) (diff)

@vladytimy
3 years ago

Removes space between line and badge on mobile devices

#2 @vladytimy
3 years ago

  • Keywords has-patch added

I couldn't test 5680.1patch with rtl style - that needs some testing.

Last edited 3 years ago by vladytimy (previous) (diff)

This ticket was mentioned in Slack in #meta by vladytimy. View the logs.


3 years ago

This ticket was mentioned in Slack in #forums by vladytimy. View the logs.


12 months ago

#5 @coffee2code
2 days ago

  • Priority changed from normal to low

Per dd32 in https://meta.trac.wordpress.org/ticket/2552#comment:24: the first item was fixed in r11568 via JavaScript, due to the recent header changes.

#6 @coffee2code
2 days ago

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

In 13383:

Support: Prevent user badge from separating from associated left vertical border at narrow viewports.

Props vladytimy, SergeyBiryukov.
Fixes #5680.

Note: See TracTickets for help on using tickets.