Make WordPress Core

Opened 3 months ago

Last modified 2 months ago

#61138 new defect (bug)

Twenty Nineteen: Latest post block text color and top/bottom spacing are not same on editor side and front-end side when we change its background.

Reported by: viralsampat's profile viralsampat Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 6.5
Component: Bundled Theme Keywords: needs-refresh
Focuses: ui, css Cc:

Description

Hello,

I have reviewed "Latest Post" block and found that when we are trying to change it background color, Its text color and top/bottom space are not same on editor side & front-end side.

I think that it should be same on both sides(Editor & Front-end)

Here, I have attached its screenshots:

Thanks,

Attachments (5)

latestpost_editor.png (233.1 KB) - added by viralsampat 3 months ago.
Back-end
latestpost_front_end.png (128.7 KB) - added by viralsampat 3 months ago.
Front-end
61138.patch (5.9 KB) - added by viralsampat 3 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.
latest-post-block-back-end.png (230.1 KB) - added by viralsampat 3 months ago.
After resolved issue Back-end:
latest-post-block-front-end.png (152.7 KB) - added by viralsampat 3 months ago.
After resolved issue front-end:

Download all attachments as: .zip

Change History (6)

@viralsampat
3 months ago

Back-end

@viralsampat
3 months ago

Front-end

@viralsampat
3 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
3 months ago

After resolved issue Back-end:

@viralsampat
3 months ago

After resolved issue front-end:

#1 @karmatosed
2 months ago

  • Keywords needs-refresh added; dev-feedback removed
  • Milestone changed from Awaiting Review to Future Release

I can confirm this is an issue. However, I think we need a different approach to the patch. First up there seem to be other things in this patch we might not need. Secondly, I think just leveling up the spacing is a gentler approach. We need to think that others might find any change a little unexpected so how can we do this as small as possible.

Note: See TracTickets for help on using tickets.