Opened 3 years ago
Last modified 8 months ago
#55183 new defect (bug)
Twenty Thirteen: Table color not reflect on the frontpage
Reported by: | umesh84 | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 5.4 |
Component: | Bundled Theme | Keywords: | has-patch dev-feedback |
Focuses: | ui, css | Cc: |
Description (last modified by )
Steps:
- Switch theme to Twenty Thirteen
- Create Table and change color after update
- Check the frontpage
Add the video Link below:
https://www.loom.com/share/6ce4c74eb3314d7bb593b0e7173e637b
Attachments (2)
Change History (8)
#3
@
2 years ago
- Description modified (diff)
- Focuses ui added
- Keywords has-patch added; needs-patch removed
- Version changed from 5.9 to 5.4
The problem is in the editor. Twenty Thirteen had removed the table cell border, but the editor replaced class names such as .editor-block-list__block
. Now the table border inherits the text color.
Changes proposed in 55183.diff:
- Use
.block-editor-block-list__block
class for table styles, while maintaining the old class name for older versions. - Set the cell padding to 0.5em to match the (editor) theme.css on the front end.
- For consideration, I added a focus outline like the editor once had used on an
.is-selected
class. Otherwise, table content can be more difficult to edit with a very light border at the top and bottom but nothing on the left or right sides.
Similar revisions would be necessary for tables in these themes:
- Twenty Twelve (which also does not show the text in a custom color—see #56466)
- Twenty Fourteen
- Twenty Seventeen
Note: See
TracTickets for help on using
tickets.
hello @sabernhardt, @umesh84
I observed that this issue is in all the themes below twenty twenty two because in the twenty twenty two we have a option available for a border and not in other themes so if we want to solved these issue we have to add the border option in other themes as well.