Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#45386 closed task (blessed) (fixed)

About: Update Freedoms page to include illustrations

Reported by: melchoyce's profile melchoyce Owned by: gziolo's profile gziolo
Milestone: 5.0 Priority: normal
Severity: normal Version:
Component: Help/About Keywords: has-patch commit fixed-5.0
Focuses: ui, administration Cc:

Description

@cathibosco has made some great illustrations of the Four Freedoms on https://wordpress.org/about/. Let's bring that over to the admin Freedoms page.

(h/t @boemedia for the suggestion!)

Attachments (4)

Freedoms.png (1.3 MB) - added by melchoyce 6 years ago.
45386.diff (6.2 KB) - added by melchoyce 6 years ago.
45386.2.diff (6.2 KB) - added by melchoyce 6 years ago.
Update "Bill of Rights" to "Freedoms"
45386.3.diff (6.9 KB) - added by gziolo 6 years ago.
Updated version with a local image

Download all attachments as: .zip

Change History (22)

@melchoyce
6 years ago

#1 @karmatosed
6 years ago

Oh my these are lovely! Thanks for adding the work here everyone.

@melchoyce
6 years ago

This ticket was mentioned in Slack in #core by melchoyce. View the logs.


6 years ago

#3 @melchoyce
6 years ago

  • Keywords has-patch needs-testing added; ui-feedback 2nd-opinion removed

#4 @tobifjellner
6 years ago

Perhaps "Bill of Rights" could simply be reduced to "Your Rights"?

@melchoyce
6 years ago

Update "Bill of Rights" to "Freedoms"

#5 @melchoyce
6 years ago

  • Summary changed from Update Freedoms page to include illustrations to About: Update Freedoms page to include illustrations

#6 @helen
6 years ago

  • Keywords commit added; needs-testing removed

#7 @gziolo
6 years ago

I see background-image: url('https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms-2x.png'); - this is the only place where we use external URL for an image in CSS files, should we add a local copy?

#8 @gziolo
6 years ago

I also noticed when talking with @afercia that the size of the image https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms-2x.png is big - 472kb. There is also https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms.png which is 112kb. I would say, let's take the smaller one and double check if we can use SVG instead to make it smaller.

@gziolo
6 years ago

Updated version with a local image

#9 follow-up: @gziolo
6 years ago

The last patch contains a local copy of https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms.png which is 112kb. It includes a few whitespace fixes for CSS file.

#10 @matveb
6 years ago

I believe optimizing the images and loading from the CDN will be done in tandem with the About Page images prior to final release.

#11 in reply to: ↑ 9 @mcsf
6 years ago

Replying to gziolo:

The last patch contains a local copy of https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms.png which is 112kb. It includes a few whitespace fixes for CSS file.

Tested by applying patch and pulling that image with:

cd /wp-admin/images
curl -O https://s.w.org/wp-content/themes/pub/wporg-main/images/freedoms.png

The page looks good on an HiDPI device (Apple MacBook Pro, recent) and in different viewports.

#12 follow-up: @gziolo
6 years ago

I believe optimizing the images and loading from the CDN will be done in tandem with the About Page images prior to final release.

I will commit the previous patch (https://core.trac.wordpress.org/attachment/ticket/45386/45386.2.diff) with CDN version of the image to make sure optimizations are applied everywhere at once.

#13 in reply to: ↑ 12 @mcsf
6 years ago

Replying to gziolo:

I believe optimizing the images and loading from the CDN will be done in tandem with the About Page images prior to final release.

I will commit the previous patch (https://core.trac.wordpress.org/attachment/ticket/45386/45386.2.diff) with CDN version of the image to make sure optimizations are applied everywhere at once.

Sounds good, go for it.

#14 @gziolo
6 years ago

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

In 43938:

About: Update Freedoms page to include illustrations.

@cathibosco has made some great illustrations of the Four Freedoms on https://wordpress.org/about/. Let's bring that over to the admin Freedoms page.

Props @melchoyce, @cathibosco, @boemedia, @karmatosed, @tobifjellner, @helen, @matveb, @mcsf.
Fixes #45386.

#15 @ocean90
6 years ago

  • Keywords fixed-5.0 added
  • Milestone changed from Awaiting Review to 5.0
  • Resolution fixed deleted
  • Status changed from closed to reopened

#16 @karmatosed
6 years ago

@cathibosco gave me the source files so adding them here if we need those for optimisation: https://www.dropbox.com/s/uugj4so5pqdtskh/Four%20frredoms.zip?dl=0 (apologies for Dropbox but they are large)

#17 @SergeyBiryukov
6 years ago

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

Merged to trunk in [44264].

This ticket was mentioned in Slack in #community-events by nao. View the logs.


5 years ago

Note: See TracTickets for help on using tickets.