mdn / mdn/fred

Consider renaming table-container to overflow-container

Open
#972 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
233
Forks
61
Avg merge
1d 1h
Merged PRs (30d)
82

Description

Summary

There are two PR opened in rari and yari addressing the problem with overflowing <math display="block"> elements. Since table-container is not exclusive to tables anymore and might be useful for any oversized content, especially on mobile, it would make sense to rename it to something more universal like overflow-container.

It’s not just a CSS selector and a class name. There’s also a wrapTables function and an extra table-container-inner wrapper.

It might also be worth revisiting the container’s behavior to make it universal for any type of content, not just tables.

Overflowing content examples

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

Start by locating the table-container selector and class, the wrapTables function, and the table-container-inner wrapper, then compare the related rari and yari pull requests. Determine the intended universal overflow behavior using the wide table and wide MathML examples; done means the naming and behavior consistently support oversized content beyond tables.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, javascript
Domain
frontend, web-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.