Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing table border when not used as the first row but the first column #20

Closed
wants to merge 1 commit into from
Closed

Fixing table border when not used as the first row but the first column #20

wants to merge 1 commit into from

Conversation

groud
Copy link

@groud groud commented Mar 17, 2016

When the is used as a column header, the first cell of each row appears rounded.
This fixes the problem.

However, as the table already has a "rounded corner attribute", I am no even sure that such rules are necessary. What do you think ?

@squidfunk
Copy link
Owner

Can you provide some example code for which the theme breaks down? This makes it easier for me to verify the issue.

@squidfunk squidfunk added this to the 0.2.2 milestone Mar 17, 2016
@squidfunk squidfunk added the bug Issue reports a bug label Mar 17, 2016
@groud
Copy link
Author

groud commented Mar 17, 2016

Here it is:
screenshot

@squidfunk
Copy link
Owner

Thanks, but I meant the markdown code (or HTML) which generates the table - would be very helpful.

@groud
Copy link
Author

groud commented Mar 17, 2016

Ah sorry, I missed the word "code". :)

Here it is:
bug.txt

@squidfunk squidfunk closed this in eade56c Mar 17, 2016
@squidfunk
Copy link
Owner

You we're right - the border-radius on the ths was not necessary - I added overflow: hidden to the table and now everything should work as expected.

@bobek bobek mentioned this pull request May 15, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue reports a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants