Get WordPress

Make WordPress Core

Opened 5 weeks ago

Closed 3 days ago

#61328 closed defect (bug) (duplicate)

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

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

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 5 weeks ago.
Back-end
verse-block-front-end.png​ (240.5 KB) - added by viralsampat 5 weeks ago.
Front-end
61328.patch​ (625 bytes) - added by viralsampat 5 weeks 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 5 weeks ago.
Back-end
after-patch-verse-block-front-end.png​ (258.3 KB) - added by viralsampat 5 weeks ago.
Front-end

Download all attachments as: .zip

Change History (11)

@viralsampat
5 weeks ago

Back-end

@viralsampat
5 weeks ago

Front-end

@viralsampat
5 weeks 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
5 weeks ago

Back-end

@viralsampat
5 weeks ago

Front-end

#1 @sabernhardt
5 weeks ago

  • Component changed from General to Bundled Theme
  • Focuses ui removed
  • Milestone changed from Awaiting Review to Future Release
  • Version 6.5.3 deleted

#2 @karmatosed
5 weeks ago

Adding to get some testing on this, thanks.

#3 @karmatosed
5 weeks ago

  • Keywords needs-testing added

#4 @karmatosed
2 weeks 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
9 days ago

#61528 was marked as a duplicate.

#6 @sabernhardt
3 days ago

  • Keywords needs-testing needs-refresh reporter-feedback removed
  • Milestone Future Release deleted
  • Resolution set to duplicate
  • Status changed from new to 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.