Making WordPress.org

Opened 5 years ago

Closed 3 years ago

#4554 closed defect (bug) (duplicate)

Formatting on canned trac reports

Reported by: pbiron's profile pbiron Owned by:
Milestone: Priority: normal
Component: Trac Keywords: close
Cc:

Description

The canned trac reports (e.g., Next Major Release) seem to not be formatting correctly.

In particular, the h2.report-results, div.paging, table.listing.tickets go full page width, whereas everything above them in constrained to have max-width: 940px; margin-left: auto; margin-right: auto.

I don't remember those things being that wide in the past...maybe they have been.

However, even if those 3 selectors are added to the CSS rule on line 219 of https://s.w.org/style/trac/wp-trac.css?118, the table still blows beyond the 940px.

Even if the table is supposed to be full width, it actually breaks on mobile (see screenshots to be added shortly).

Attachments (2)

trac-report.png (143.3 KB) - added by pbiron 5 years ago.
trac-report-mobile.png (89.9 KB) - added by pbiron 5 years ago.
this in Chrome's "mobile preview" in the inspector.

Download all attachments as: .zip

Change History (9)

@pbiron
5 years ago

@pbiron
5 years ago

this in Chrome's "mobile preview" in the inspector.

#1 @SergeyBiryukov
5 years ago

Even if the table is supposed to be full width, it actually breaks on mobile (see screenshots to be added shortly).

IIRC, the table is indeed supposed to be full width. We could try to improve the mobile styling though.

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


5 years ago

#3 @tellyworth
5 years ago

  • Keywords close added

Even if we fix the styling, there's too much info on that page for it to ever really be mobile friendly. Is there anything we can do here that has enough benefit to be worth spending time on it?

#4 @dd32
5 years ago

Is there anything we can do here that has enough benefit to be worth spending time on it?

After a very quick look, I can't see anything obvious we can do here.

I don't think removing the mobile-width of the header/footer would be a wise move here either, as at least users can zoom to the left hand side to get a proper view, and then scroll for the rest.

I would suggest that this is probably a wontfix until we reskin (again) or more likely switch to a non-Trac system.

#6 @vladytimy
3 years ago

There is a proposal here (3rd item) for responsive Trac reports:

https://meta.trac.wordpress.org/ticket/4917#comment:17

This is a 4 part patch that tries to bring better responsiveness for Trac reports.
Your feedback and testing would be much appreciated. Thank you!

#7 @dd32
3 years ago

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

This might be improved with [11610], but regardless, I'm going to close this as a duplicate of #4917

Note: See TracTickets for help on using tickets.