#57087 closed defect (bug) (fixed)
Twenty Twenty: Button text is not visible in footer widget
Reported by: | mayur8991 | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-screenshots has-patch has-testing-info commit |
Focuses: | css | Cc: |
Description
- Choose Twenty Twenty Theme
- Activate theme
- Go to widget section
- add button in footer widget and add texts then save it
in frontend that button text is not visible
Attachments (7)
Change History (23)
#1
@
2 years ago
- Keywords has-patch added
Css issue
you can remove .footer-widgets a from function.php line no 755
OR
you can add custom css for button color
Ex:
.widget.widget_block .wp-element-button{color:#ffffff;}
#2
@
2 years ago
I was able to reproduce the issue. Tested the patch and it works fine. Not sure if it is the right way.
#3
@
2 years ago
- Component changed from Themes to Bundled Theme
- Focuses css added
- Keywords needs-testing added
- Summary changed from Twenty Twenty Theme - Button texts is not visible in footer widget to Twenty Twenty: Button text is not visible in footer widget
- Version 6.1 deleted
The .footer-widgets a
selector has been in the theme since PR 447. However, links have a default color regardless of location, so this might be safe to remove. This would need testing with many different links in widgets.
One quirk is that changing the Primary (or accent) color in the Customizer will turn the button text color to white. This color continues to apply when switching back to the default Primary color.
.header-footer-group .wp-block-button:not(.is-style-outline) .wp-block-button__link { color: #ffffff; }
In WordPress 6.0, the button text was also pink, unless the link was visited earlier. (The editor styles had set a white color for the :visited
state.)
#4
@
21 months ago
- Version set to 6.2
I have applied the patch in svn and tested it with WordPress 6.2, the patch works perfectly.
#5
@
20 months ago
Patch works fine in WordPress 6.2
when theme will be available with the patch ?? @sabernhardt
This ticket was mentioned in Slack in #core by mayur8991. View the logs.
19 months ago
#7
@
19 months ago
Hi
Test report for - https://core.trac.wordpress.org/attachment/ticket/57087/57087.diff patch
Environment:
WordPress - V6.2.2
Theme - Twenty Twenty
Os: Mac
Browser: Chrome
Expected Result: The button text added from the widget should be visible on the frontend ☑️
Before Screenshot:
Backend widget - https://prnt.sc/i6eLFO3PLq3G
Frontend Footer - https://prnt.sc/T3D0YgtcnP3x
After Screenshot:
Backend widget - https://prnt.sc/i6eLFO3PLq3G
Frontend Footer - https://prnt.sc/XeoTFpPVhZvS
Thanks!
This ticket was mentioned in PR #4564 on WordPress/wordpress-develop by @mukesh27.
19 months ago
#8
Trac ticket: https://core.trac.wordpress.org/ticket/57087
#9
@
19 months ago
- Milestone changed from Awaiting Review to 6.3
- Version 6.2 deleted
@sabernhardt Your suggestion look good and solve the issue in my testing.
Could you take a look the PR?
Moving it to 6.3
consideration.
#10
@
18 months ago
Expected Result: Button text should visible in footer widget same as backend
Before Patch Screenshot: https://prnt.sc/WBVVcmvEz5E6
After Patch Screenshot: https://prnt.sc/Sc1wWrkFodMi
#11
@
18 months ago
The white text from .header-footer-group .wp-block-button:not(.is-style-outline) .wp-block-button__link
could be considered another issue, not an ideal solution. It is lighter than the beige for a Button link in the content area, and it already occurs if you choose another Accent color in the Customizer (plus it remains even after restoring the default Accent color). Then choosing one of the theme colors for the text would have no effect on the front end.
I'm more concerned about the unreadable default colors, though I tried fixing both situations with the :where()
selector to avoid increasing specificity.
#12
@
18 months ago
Yes @sabernhardt, this is better solution(Without using specific color) and Patch is working fine.
#13
@
18 months ago
- Keywords has-testing-info added
Test Report for https://core.trac.wordpress.org/attachment/ticket/57087/57087.3.diff
Environment:
WordPress - 6.3-alpha
Theme - Twenty Twenty
Os: Windows
Browser: Chrome
Before Patch : https://prnt.sc/FBxweA4PGpXn
After Patch : https://prnt.sc/1qngccz3tygm
Result After Patch : The button text is visible on the frontend
https://core.trac.wordpress.org/attachment/ticket/57087/57087.3.diff is working as expected
Thanks.
#14
@
18 months ago
- Keywords commit added; needs-testing removed
- Owner set to audrasjb
- Status changed from new to accepted
Works fine on my side too, thanks everyone.
@mukesh27 commented on PR #4564:
16 months ago
#16
The ticket already merged.
added screenshot for better understanding