Opened 6 years ago
Closed 4 years ago
#45876 closed defect (bug) (fixed)
Twenty Nineteen: Dropcap appears higher than expected in Firefox
Reported by: | kjellr | Owned by: | ianbelanger |
---|---|---|---|
Milestone: | 5.5 | Priority: | normal |
Severity: | normal | Version: | 5.0 |
Component: | Bundled Theme | Keywords: | has-screenshots has-patch |
Focuses: | css | Cc: |
Description
As noted by @josephdickson, Firefox renders the dropcap text roughly 0.2em higher than in Chromium:
@josephdickson proposed a possible solution:
With a little help from @electmikemann ( via Twitter) I found a solution that I'd like to propose for a future update.
The @-moz prefix targets Firefox and is ignored by other browsers. I've tested it via the Customizer's Additional CSS box. Anyone can do the same for a quick fix.
@-moz-document url-prefix() { .entry .entry-content .has-drop-cap:not(:focus)::first-letter { margin-top: 0.2em; } }
---
This issue was migrated over from Twenty Nineteen's GitHub repository:
Attachments (4)
Change History (18)
#2
@
6 years ago
- Keywords has-screenshots added; needs-testing removed
- Milestone changed from Awaiting Review to Future Release
#3
@
5 years ago
- Focuses css added
- Milestone changed from Future Release to 5.5
- Version changed from 5.0.2 to 5.0
#4
@
5 years ago
I propose this patch on the basis of the hack proposed on this ticket. This refers to the scss style-editor and _blocks files and the generated css files.
#5
@
5 years ago
- Keywords has-patch needs-refresh added; needs-patch removed
Hi @maxpertici,
Thanks for your patch. It does seem to fix the issue in Firefox, but I noticed that there is a change to the package.json
file in your diff. Please remove this change and resubmit the patch.
#6
@
5 years ago
Hi @ianbelanger,
I hope this one will be better.
Sorry for the setback. Obviously a change on the grunt config.
#8
@
4 years ago
I tested this issue on the latest Linux (Fedora) Firefox and Chromium and this fix works well. I hope a commit can be made soon :)
#9
@
4 years ago
- Keywords needs-testing added
The last patch no longer worked for me, it would not apply anymore. So I have updated the patch. @samful would you mind testing it out. Works good for me in chrome and firefox. Thanks
#10
@
4 years ago
That patch is much tidier @ianbelanger. It all seems to work for me, I'll add a screen-shot below of before and after:
#11
@
4 years ago
- Keywords commit added; needs-testing removed
- Owner set to ianbelanger
- Status changed from new to reviewing
Reviewing for commit.
@kjellr Thanks for migrating this request.