Opened 8 years ago
Closed 6 years ago
#45876 closed defect (bug) (fixed)
Twenty Nineteen: Dropcap appears higher than expected in Firefox
| Reported by: | kjellr | Owned by: | ianbelanger |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.5 |
| Component: | Bundled Theme | Version: | 5.0 |
| Severity: | normal | Keywords: | has-screenshots has-patch |
| Cc: | Focuses: | css |
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
@
8 years ago
- Keywords has-screenshots added; needs-testing removed
- Milestone Awaiting Review → Future Release
#4
@
6 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
@
6 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
@
6 years ago
Hi @ianbelanger,
I hope this one will be better.
Sorry for the setback. Obviously a change on the grunt config.
#8
@
6 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
@
6 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
@
6 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
@
6 years ago
- Keywords commit added; needs-testing removed
- Owner set to
- Status new → reviewing
Reviewing for commit.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

@kjellr Thanks for migrating this request.