Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#61328 closed defect (bug) (duplicate)

Twenty Fifteen: Verse block font family is different in back-end & Front-end.

Reported by: viralsampat Owned by:
Priority: normal Milestone:
Component: Bundled Theme Version:
Severity: normal Keywords: has-screenshots has-patch
Cc: Focuses: css

Description

Hello,

I have reviewed "Verse" block and found that its font-family is different in both(Editor & Front-end) sides.

I think that it should be same on both sides.

I have checked code trac and found same issue on Twenty Seventeen: #61140

Thanks,

Attachments (5)

verse-block-back-end.png​ (281.7 KB ) - added by viralsampat 2 years ago.
Back-end
verse-block-front-end.png​ (240.5 KB ) - added by viralsampat 2 years ago.
Front-end
61328.patch​ (625 bytes ) - added by viralsampat 2 years ago.
I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.
after-patch-verse-block-back-end.png​ (192.0 KB ) - added by viralsampat 2 years ago.
Back-end
after-patch-verse-block-front-end.png​ (258.3 KB ) - added by viralsampat 2 years ago.
Front-end

Download all attachments as: .zip

Change History (11)

@viralsampat
2 years ago

Back-end

@viralsampat
2 years ago

Front-end

@viralsampat
2 years ago

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

#1 @sabernhardt
2 years ago

  • Component General → Bundled Theme
  • Focuses ui removed
  • Milestone Awaiting Review → Future Release
  • Version 6.5.3

#2 @karmatosed
2 years ago

Adding to get some testing on this, thanks.

#3 @karmatosed
2 years ago

  • Keywords needs-testing added

#4 @karmatosed
2 years ago

  • Keywords needs-refresh reporter-feedback added; dev-feedback removed

@viralsampat in testing for me this doesn't unify the fonts on the front and back. Are you still seeing this patch working to do do that?

#5 @sabernhardt
2 years ago

#61528 was marked as a duplicate.

#6 @sabernhardt
2 years ago

  • Keywords needs-testing needs-refresh reporter-feedback removed
  • Milestone Future Release
  • Resolution → duplicate
  • Status new → closed

The patch works for me, but I combined that with similar changes for the other six themes on #61140 so the discussion (and decision for all seven) stays on one ticket.

Note: See TracTickets for help on using tickets.