#8175 closed defect (bug) (fixed)
Insert/edit link dialog missing primary action button (OK/Add Link)
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Component: | Support Forums | Keywords: | has-patch |
| Cc: |
Description
The "OK" or "Add Link" button in the TinyMCE link dialog is invisible/unstyled (ghost button), likely a CSS specificity issue where the site's styles override the button's background. Only "Cancel" is visible.
Attachments (1)
Change History (5)
This ticket was mentioned in PR #554 on WordPress/wordpress.org by @yani.iliev.
8 weeks ago
#1
- Keywords has-patch added
Closes: https://meta.trac.wordpress.org/ticket/8175
The --wp-admin-theme-color custom property is declared on :root referencing
var(--wp--preset--color--blueberry-1), but that variable is only defined on
body by WordPress core's global-styles-inline-css. Since :root (<html>)
is the parent of body, the referenced variable is not available in that scope.
This changes the declaration from :root to body so both variables are
defined on the same element and the reference resolves correctly.
Note: I used AI to generate the PR description and title. The source code has been reviewed and tested by me locally before opening this PR.
#2
@
8 weeks ago
- Owner set to dd32
- Resolution set to fixed
- Status changed from new to closed
In 14666:
#3
@
8 weeks ago
Beautiful. I confirmed it is fixed. Smart solution, dd32, to include both :root, body. Much better!
#4
@
8 weeks ago
@yaniiliev Thanks for the PR!
I included :root as well because it seems to be where it's defined / used in other places, but global-styles indeed only sets it the colour body. Fallback included for :root() just incase, because I didn't want to deal with this elsewhere.
The darker colour was required for the hover state, without it the button vanished on hover :)
Classic editor missing primary link/button.