Opened 9 years ago
Closed 9 years ago
#38447 closed defect (bug) (fixed)
Twenty Seventeen: Formatting in table
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | ui-feedback has-screenshots has-patch |
Focuses: | Cc: |
Attachments (5)
Change History (15)
#3
@
9 years ago
I would like to work on this one, but @karmatosed what exactly is the problem here? It looks okay to me.
#4
@
9 years ago
That's awesome @kamena. As per @melchoyce's comment, do you see the left padding in the cells?
#6
@
9 years ago
First of all, @snacking welcome here, I see it's your first commit
About the patch - from the comment above "Yeah, shouldn't be any left-padding in cells — will take a look."
Meaning - you should still be having the right padding. The issue of this can be seen in the mobile view screenshot you posted.
@
9 years ago
This is the updated fix according to the note by the last comment with the padding removed only from the left side
#8
@
9 years ago
To clarify, is this happening on desktop, mobile or both? I'm not seeing it in Chrome, Firefox or Safari.
#9
@
9 years ago
I created a riff on @snacking's 38447.2.diff - it:
- Removes the left padding on the first cell in a row, and the right padding on the last cell in each row - the padding is still there between cells, so the contents don't meet.
- Does the opposite for RTL.
- Updates the editor styles (which didn't include the most up-to-date editor styles).
Yeah, shouldn't be any left-padding in cells — will take a look.