Making WordPress.org

Opened 7 years ago

Last modified 4 years ago

#2822 new enhancement

Support forum should have better styling on mobile (6.44′ screen)

Reported by: alkesh7's profile alkesh7 Owned by:
Milestone: Priority: low
Component: Support Forums Keywords: needs-patch needs-ui
Cc:

Description

Hi guys,

I noticed some issues with wordpress.org support forum with my Mi Max smartphone (screens size - 6.444').

Gravatar image showing outside the content, http://prnt.sc/eu2eey

There are lots of issues, not only single one that I have listed above. Someone need to play with the same smartphone and access whole wordpress.org site

Regards,
Alkesh

Attachments (5)

2822.diff (1.7 KB) - added by dufresnesteven 5 years ago.
Improve: Pagination control on smaller viewports.
current.png (147.3 KB) - added by dufresnesteven 5 years ago.
Current layout of the pagination component.
Updated-layout.png (126.3 KB) - added by dufresnesteven 5 years ago.
Updated Layout
2822.pagination-count-misaligned.png (6.0 KB) - added by SergeyBiryukov 5 years ago.
reg_fix.diff (623 bytes) - added by dufresnesteven 5 years ago.
Patch to fix regression of "Viewing" alignment.

Download all attachments as: .zip

Change History (19)

#1 @ocean90
7 years ago

  • Component changed from Browse Happy to Support Forums
  • Priority changed from normal to low

#2 @melchoyce
7 years ago

  • Keywords needs-ui added

#3 @kjellr
5 years ago

👋 I spent a little time thinking about some light adjustments to help the forums behave better on mobile. Here are some comps along with some specific notes:


Landing Page:

  • The Create Issue / Pagination should be stacked.
  • Pagination should be wider and (at least a little) taller to be more amenable to touch interfaces.
  • The entire content width could be widened a bit to take advantage of the smaller available space.
  • The "Pinned" and "Resolved" icons should hang off to the left of the text, rather than simply being inline.
  • On the smallest screens (320ish), the table headings do not line up with the content. We should rework to include those inline instead.

https://cldup.com/ZSDkoJrZK8-3000x3000.png


Individual Topic

  • In general, tighten up margins to make better use of space.
  • Like the landing page, we should widen the content area a little bit.
  • Reduce the font size of the title, for better line lengths.
  • We should tighten the extra-tall line height for the breadcrumb.
  • Make the avatars smaller and stack the "Resolved" tag.
  • Add larger body font sizes for improved readability.

https://cldup.com/HmlViDClRm-2000x2000.png

#4 @melchoyce
5 years ago

These look great, @kjellr 👍

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


5 years ago

@dufresnesteven
5 years ago

Improve: Pagination control on smaller viewports.

@dufresnesteven
5 years ago

Current layout of the pagination component.

@dufresnesteven
5 years ago

Updated Layout

#6 @dufresnesteven
5 years ago

I updated the pagination control to be more mobile friendly in: 2822.diff

Images above.

Regression Testing Matrix

OS Browser Support
WIndows 7 IE 9
Window 8 IE 10
Window 8 IE 11
Window 8.1 IE 11
Window 10 IE 11
Window 10 Edge
Android Galaxy 9 Firefox
MacOS Mojave Safari 12.1

#7 @dd32
5 years ago

In 9620:

Support: Better styling for pagination on smaller screen widths.

Props dufresnesteven, kjellr.
See #2822.

#8 follow-up: @SergeyBiryukov
5 years ago

[9620] has introduced a regression in replies pagination inside a topic, as seen in this topic for example.

The replies count is overlapping with the page links, see 2822.pagination-count-misaligned.png. Previously, it used to be aligned to the left.

#9 in reply to: ↑ 8 @dufresnesteven
5 years ago

Replying to SergeyBiryukov:

[9620] has introduced a regression in replies pagination inside a topic, as seen in this topic for example.

The replies count is overlapping with the page links, see 2822.pagination-count-misaligned.png. Previously, it used to be aligned to the left.

Great catch. I'll submit a patch.

@dufresnesteven
5 years ago

Patch to fix regression of "Viewing" alignment.

#10 @dd32
5 years ago

In 9696:

Support: Fix the alignment of the "Viewing" text inside topics when multiple pages exist.

Props dufresnesteven, SergeyBiryukov.
See #2822.

#11 @dd32
5 years ago

In 9697:

Support: Restore some styles removed in [9696] by adding a missing SCSS file from [9664].

See #3659, #2822.

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


4 years ago

#13 @Clorith
4 years ago

Are there any steps missing from this ticket, I see it still has the needs-ui flag, is this something actively being worked / in the pipeline still, @kjellr ?

#14 @kjellr
4 years ago

I haven't kept up on this ticket, and I'm not actively working on this personally. It looks like the only thing that was implemented from my recommendations a couple years ago is the pagination. There are still a number of other bullet point recommendations I made above that I think could still be implemented if someone has the time.

Note: See TracTickets for help on using tickets.