Opened 17 months ago
Last modified 18 hours ago
#60336 new defect (bug)
Twenty Sixteen: Table block having issue with border when text color is used.
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | 6.4.2 |
Component: | Bundled Theme | Keywords: | has-patch has-test-info changes-requested |
Focuses: | Cc: |
Description
Steps to reproduced the issue :-
- Activate Twenty Sixteen theme.
- Choose table block.
- Add some text.
- Apply text color.
Now you can able to see that the choosen text color also applied on border in editor side but in the user side the border color remains same as default border color.
I have attached video for better understanding.
Video URL :- https://share.cleanshot.com/LJlVp3460bvvl5ZYRTsP
Attachments (2)
Change History (15)
#3
@
16 months ago
Testing Report for: https://core.trac.wordpress.org/attachment/ticket/60336/60336.patch ✅
Environment:
PHP: 8.1.9
WordPress: 6.4.3
Theme: Twenty Sixteen
Browser: Chrome
Plugin: No plugins
Before Patch Screenshot:
Backend: https://prnt.sc/xW62ZwC-rnox
Frontend: https://prnt.sc/0TqB0D2gLTgB
After Patch Screenshot:
Backend: https://prnt.sc/xW62ZwC-rnox
Frontend: https://prnt.sc/NLy4wPuQ_UKn
✅ Issue resolved with a patch.
#4
@
16 months ago
Edit: I did not apply the patch correctly. Here is my updated test report.
Testing Report for: https://core.trac.wordpress.org/attachment/ticket/60336/60336.patch
Environment
- WordPress: 6.4.3
- PHP: 8.1.23
- Server: nginx/1.16.0
- Database: mysqli (Server: 8.0.16 / Client: mysqlnd 8.1.23)
- Browser: Chrome 121.0.0.0
- OS: macOS
- Theme: Twenty Sixteen 3.1
- MU Plugins: None activated
- Plugins:
- Test Reports 1.1.0
Actual Results
- ❌ The table block that does not have a text color set (the default block as it is inserted), has the wrong border styles on the front. The default needs to be unchanged.
- ✅ The table block that has a text color shows the correct border styles in the editor and front.
#5
follow-up:
↓ 6
@
16 months ago
There is another bug that is not mentioned in the ticket description:
Before the patch, the border color in the editor and front does not match, even when the user does not select a text color.
So it would be good to fix that bug, too. -The editor should match the color that is the default color on the front.
#6
in reply to:
↑ 5
@
16 months ago
Replying to poena:
There is another bug that is not mentioned in the ticket description:
Before the patch, the border color in the editor and front does not match, even when the user does not select a text color.
So it would be good to fix that bug, too. -The editor should match the color that is the default color on the front.
@poena I have tested it with patch, and it does fix table default color issue too.
#7
@
16 months ago
Hi
@poena I have tested it with patch, and it does fix table default color issue too.
The color of the border in both the editor and front should be #d1d1d1
unless the user changes the text color.
The colors should not only match: the table needs to keep the original color that is part of the theme design. The patch removes the color.
That is what I meant by "The default needs to be unchanged." and "The editor should match the color that is the default color on the front."
#8
@
15 months ago
- Keywords changes-requested added
- Milestone changed from Awaiting Review to Future Release
If I am to understand correctly whilst this patch does resolve the issue there is a recommendation on improving it. It would be great to see that added so we can progress this for everyone, thank you.
#12
@
5 days ago
- Summary changed from Twenty Sixteen :- Table block having issue with border when text color is used. to Twenty Sixteen: Table block having issue with border when text color is used.
This ticket was mentioned in PR #9058 on WordPress/wordpress-develop by @sandeepdahiya.
18 hours ago
#13
Trac ticket: https://core.trac.wordpress.org/ticket/60336
Tables with custom text color now have matching borders on the frontend— by aligning border-color with currentColor. This creates a more cohesive and visually consistent block editor experience while preserving the default #d1d1d1 fallback when no custom color is set.
After patch when color is applied to text - backend and front end
After patch when no color is applied to text - border color is theme default color #d1d1d1
Thanks for your work on this patch.
I've tested the patch, and it appears to be working as intended on my end. Attaching a screenshot of the same.
Environment