Opened 6 years ago
Closed 4 years ago
#45198 closed defect (bug) (fixed)
"Remove featured image" link should be in Red Color
Reported by: | monikarao | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.8 | Priority: | normal |
Severity: | normal | Version: | 4.9.8 |
Component: | Post Thumbnails | Keywords: | has-patch commit has-testing-info |
Focuses: | ui, administration | Cc: |
Description
All delete links are appearing in Red Color except "Remove featured image" so it should be in Red color.
Attachments (5)
Change History (13)
#4
follow-up:
↓ 6
@
4 years ago
45198.1.2.diff correctly matches the "Remove featured image" link to the red used for the "Move to Trash" link and other destructive actions, both in the original state and when hovering the cursor over it. Those links are also the same red in all nine standard color schemes.
Tested with Firefox 88 and Windows 10
It would be nice to make that link a button instead, but that's probably outside the scope of this ticket.
#5
@
4 years ago
- Keywords commit added; needs-testing removed
45198.1.2.diff correctly fixes the reported problem with the link color in the classic editor and does not impact the block editor.
Testing Environment:
- OS: MacOS
- Browser: Chrome v90.0.4430.93 and Firefox v88.0
- Localhost: Docker and WP Local
- Plugins: Classic Editor activated
- Theme: Twenty Twenty-One
Marking for commit
.
#6
in reply to:
↑ 4
@
4 years ago
Replying to sabernhardt:
It would be nice to make that link a button instead, but that's probably outside the scope of this ticket.
I agree it would be better as a button instead of a link. I too think the button enhancement is out of the scope of this ticket. There may be other links in the classic editor that could be changed to a button.
#7
@
4 years ago
- Keywords has-testing-info added
How to test
Reproduce the issue
Before applying the patch, follow these steps to reproduce the issue:
- Pull the latest changes from
wordpress-develop
repo - Run the build process as noted in the Getting Started document
- Install and activate the Classic Editor plugin
- Navigate to Pages > Sample Page and open the page
- Add a feature image
- Update to save it
- Notice the color of the "Remove featured image" link
Test the fix
- Apply the patch:
npm run grunt patch:45198
- Repeat the rest of the steps from above (expect for the 1st step)
Currently it's appearing in Blue Color