Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#48386 closed task (blessed) (fixed)

Update Twenty Twenty with Latest Changes for RC 2

Reported by: ianbelanger's profile ianbelanger Owned by: ianbelanger's profile ianbelanger
Milestone: 5.3 Priority: normal
Severity: normal Version: 5.3
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

This is the reference ticket for the Twenty Twenty theme which is currently under development, ready for 5.3 RC 2.

Initial development is happening on GitHub: https://github.com/WordPress/twentytwenty

Read the announcement post: https://make.wordpress.org/core/2019/09/06/introducing-twenty-twenty/

Attachments (11)

48386.diff (44.6 KB) - added by ianbelanger 5 years ago.
2020-landscape-1.png (1.2 KB) - added by ianbelanger 5 years ago.
2020-landscape-2.png (6.1 KB) - added by ianbelanger 5 years ago.
2020-square-1.png (4.1 KB) - added by ianbelanger 5 years ago.
2020-square-2.png (3.3 KB) - added by ianbelanger 5 years ago.
2020-three-quarters-1.png (2.3 KB) - added by ianbelanger 5 years ago.
2020-three-quarters-2.png (2.6 KB) - added by ianbelanger 5 years ago.
2020-three-quarters-3.png (25.5 KB) - added by ianbelanger 5 years ago.
2020-three-quarters-4.png (2.1 KB) - added by ianbelanger 5 years ago.
missing-images.png (41.0 KB) - added by desrosj 5 years ago.
48386.1.diff (53.3 KB) - added by ianbelanger 5 years ago.
should be gtg, includes svn add for images and 2 fonts that names were changed in

Download all attachments as: .zip

Change History (31)

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


5 years ago

@ianbelanger
5 years ago

#2 @ianbelanger
5 years ago

There are 8 images that need to be added via svn add for the starter content. I will upload them here.

#3 @ianbelanger
5 years ago

They are contained within the assets/images folder

#4 @desrosj
5 years ago

  • Keywords i18n-change added

48386.diff looks good to me.

There is one string change, so I am tagging as i18n-change.

I did notice that in the starter content, there are two images not loading. They do not appear to be in the assets/images folder on GitHub.

@ianbelanger
5 years ago

should be gtg, includes svn add for images and 2 fonts that names were changed in

#5 @ianbelanger
5 years ago

  • Keywords has-patch added

#6 @desrosj
5 years ago

  • Keywords commit dev-feedback added
  • Version set to trunk

This looks good to me. I'd love one more set of committer eyes.

#7 @SergeyBiryukov
5 years ago

  • Keywords dev-reviewed added; dev-feedback removed

48386.1.diff looks good to me as well.

#8 @desrosj
5 years ago

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

In 46571:

Bundled Themes: Update Twenty Twenty.

This brings trunk’s version of Twenty Twenty in-sync with GitHub.

For a complete list of changes since [46551], see https://github.com/WordPress/twentytwenty/compare/bc89c51...4549fd9.

Reviewed by SergeyBiryukov, desrosj, ianbelanger.
Props anlino, ianbelanger, poena, williampatton, nielslange, acosmin, intimez, itowhid06, decrecementofeliz, aristath, westonruter, mukesh27, jorgefilipecosta, karmatosed.
Fixes #48386.

#9 @afercia
5 years ago

  • Keywords needs-patch added; i18n-change has-patch commit dev-reviewed removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

During latest accessibility bug-scrub we discovered a bug related to the comments "Reply" link that makes the comments pages barely usable for all users.

WordPress is still in RC 2 and the fix will need to be synced from the GitHub repo to core, so I'm reopening this ticket instead of creating a new one.

The bug is detailed on https://github.com/WordPress/twentytwenty/issues/927

The fix appears to be super simple and I'd strongly recommend to fix it before final release. I do realize it's RC 2 time but seems to me Twenty Twenty can't be released with this bug unsolved.

I guess this implies a quick RC 3 will be necessary.

#10 @ianbelanger
5 years ago

  • Keywords has-patch added; needs-patch removed
  • Resolution set to fixed
  • Status changed from reopened to closed

The aforementioned bug https://github.com/WordPress/twentytwenty/issues/927 has been resolved and will be fixed in RC 3. Closing this in favor of a new ticket for RC 3

#11 @desrosj
5 years ago

In 46668:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since 5.3 RC3, see https://github.com/WordPress/twentytwenty/compare/dea9290...c267289.

Props williampatton, poena, andersnoren.
See #48110, #48386, #48450, #48293.
Fixes #48505.

#12 @desrosj
5 years ago

In 46669:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since 5.3 RC3, see https://github.com/WordPress/twentytwenty/compare/dea9290...c267289.

Props williampatton, poena, andersnoren.
See #48110, #48386, #48450, #48293.
Fixes #48505.

#13 @SergeyBiryukov
5 years ago

In 46701:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since 5.3 RC4, see https://github.com/WordPress/twentytwenty/compare/c267289...898792b.

Props williampatton, poena, andersnoren, desrosj, schlessera, leprincenoir, alextran, aristath, b-07.
See #48110, #48386, #48450, #48505.
Fixes #48557.

#14 @SergeyBiryukov
5 years ago

In 46712:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since 5.3 RC4, see https://github.com/WordPress/twentytwenty/compare/c267289...ab5fa23.

Reviewed by SergeyBiryukov.
Props williampatton, poena, andersnoren, desrosj, schlessera, leprincenoir, alextran, aristath, b-07, allancole, karmatosed, pierlo, joen, ianbelanger.
See #48110, #48386, #48450, #48505.
Merges [46701], [46702], [46711] to the 5.3 branch.
Fixes #48557.

#15 @SergeyBiryukov
5 years ago

In 46713:

Bundled Themes: Update Twenty Twenty.

  • Add a11y-friendly default text-colors when selecting a background-color.
  • Remove margins for the group block when a background-color is set.
  • Excluding pullquote block from width resize rules.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46711], see https://github.com/WordPress/twentytwenty/compare/ab5fa23...162edc0.

Props allancole, karmatosed.
See #48110, #48386, #48450, #48505.
Fixes #48557.

#16 @SergeyBiryukov
5 years ago

In 46714:

Bundled Themes: Update Twenty Twenty.

  • Add a11y-friendly default text-colors when selecting a background-color.
  • Remove margins for the group block when a background-color is set.
  • Excluding pullquote block from width resize rules.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46711], see https://github.com/WordPress/twentytwenty/compare/ab5fa23...162edc0.

Reviewed by azaozz, SergeyBiryukov.
Props allancole, karmatosed.
See #48110, #48386, #48450, #48505.
Merges [46713] to the 5.3 branch.
Fixes #48557.

#17 @SergeyBiryukov
5 years ago

In 46715:

Bundled Themes: Update Twenty Twenty.

  • Add .has-background excluding condition to wide and full group-blocks.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46714], see https://github.com/WordPress/twentytwenty/compare/162edc0...fa8c3fc.

Props allancole, karmatosed.
See #48110, #48386, #48450, #48505.
Fixes #48557.

#18 @SergeyBiryukov
5 years ago

In 46716:

Bundled Themes: Update Twenty Twenty.

  • Add .has-background excluding condition to wide and full group-blocks.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46714], see https://github.com/WordPress/twentytwenty/compare/162edc0...fa8c3fc.

Reviewed by azaozz, SergeyBiryukov.
Props allancole, karmatosed.
See #48110, #48386, #48450, #48505.
Merges [46715] to the 5.3 branch.
Fixes #48557.

#19 @SergeyBiryukov
5 years ago

In 46722:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46715], see https://github.com/WordPress/twentytwenty/compare/fa8c3fc...a9e73ab.

Props Anlino, karmatosed, williampatton, poena, joyously, ianbelanger, chetan200891.
See #48110, #48386, #48450, #48505, #48557.
Fixes #48571.

#20 @SergeyBiryukov
5 years ago

In 46723:

Bundled Themes: Update Twenty Twenty.

This brings Twenty Twenty in sync with GitHub. For a full list of changes since [46715], see https://github.com/WordPress/twentytwenty/compare/fa8c3fc...a9e73ab.

Props Anlino, karmatosed, williampatton, poena, joyously, ianbelanger, chetan200891.
See #48110, #48386, #48450, #48505, #48557.
Merges [46722] to the 5.3 branch.
Fixes #48571.

Note: See TracTickets for help on using tickets.