twbs / twbs/bootstrap

Support tables in cards (Gist provided)

Open
#39,774 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature
Dominant language
MDX
Stars
175k
Forks
78.6k
Avg merge
7h 19m
Merged PRs (30d)
35

Description

Prerequisites
Proposal

Hello,

I'd like to suggest to officially support tables inside of cards. I'm aware that this comes with a lot of edge cases to consider. Cards are a great way to display mixed content or even just work as a visual container. Tables should be one element, as it allows to annotate tables with additional context in the same visual container. There's another ticket #17395, but that one's locked, so I cannot add a comment to that.

I tried to come up with a fix and I think it is pretty clean and catches most of the cases a user could have. At least in my project, I couldn't find a combination that doesn't work with my snippet.

Please have a look at my gist:
https://gist.github.com/tinyoverflow/999c7797ce51582ea4481b2501e7ad20

Motivation and context

In my current project, I make heavy use of tables inside of cards. They don't look good by default. Researching this topic resulted in a lot of people with this issue, so I decided to try to tackle this one.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Start by reviewing the provided Gist and the related issue #17395, then inspect the existing card and table styling; done should mean tables display appropriately inside cards, including the edge cases covered by the agreed approach.

Written by the indexing model from the issue text.

Assessment

Tech stack
bootstrap, css, html
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.