Making WordPress.org

Opened 19 months ago

Last modified 2 months ago

#7775 new enhancement

VCS: Changeset pages have some text with small font size, which is less legible

Reported by: pitamdey's profile pitamdey Owned by:
Milestone: Priority: lowest
Component: Trac Keywords: has-screenshots
Cc:

Description

URL : https://core.trac.wordpress.org/changeset/58910

The small font size on the page affects readability, increasing it for better visibility would be helpful.

Attachments (3)

Screenshot 2024-09-17 at 6.04.32 PM.png (370.5 KB) - added by pitamdey 19 months ago.
Screenshot for better understanding
7775.diff (1.5 KB) - added by viralsampat 19 months ago.
I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.
meta_contribution.png (381.8 KB) - added by viralsampat 19 months ago.
Screenshot:

Download all attachments as: .zip

Change History (6)

@pitamdey
19 months ago

Screenshot for better understanding

#1 @ryelle
19 months ago

  • Component changed from Make (Get Involved) / P2 to Trac

@viralsampat
19 months ago

I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.

@viralsampat
19 months ago

Screenshot:

#2 @dd32
17 months ago

  • Priority changed from low to lowest

For reference, the files edited in the diff above are upstream trac CSS files, which should not be edited directly. Ideally, any customisations should go in wp-trac.css for easier trac upgrades.

I am inclined not to merge this change though; and instead upgrade the Trac CSS/JS we're loading, which appears to be severely out-of-date. These font-sizes were increased in https://trac.edgewall.org/changeset/15523 which is in Trac 1.3, which we're likely to upgrade to shortly.

Upgrading the CSS/JS will however likely break other CSS/JS customisations we've made to trac, so it's not an urgent task.

#3 @chealer
2 months ago

  • Summary changed from The page has a small font size which is not properly visible to VCS: Changeset pages have some text with small font size, which is less legible
  • Type changed from defect (bug) to enhancement

I do not consider this as a bug. The elements you highlight do have a small font size, but remain easy to read for me, and people with lower vision can simply zoom in. These elements were presumably set to a smaller size to save space for the variable parts of content. As these elements are the same on all commit pages, developers rarely need to read them, so it seems perfectly acceptable to me that reading them at default zoom level is less comfortable than reading the commit message or contents.

This comment and any other from myself on this ticket is offered under the terms of CC0 1.0.

Note: See TracTickets for help on using tickets.