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

Update flexbox.json for Safari bugs #6730

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

aardrian
Copy link
Contributor

Added information about Safari/iOS issues with tables that use display:flex. Includes link to Safari bug https://bugs.webkit.org/show_bug.cgi?id=257458.

Added information about Safari/iOS issues with tables that use `display:flex`. Includes link to Safari bug https://bugs.webkit.org/show_bug.cgi?id=257458.
@jensimmons
Copy link
Contributor

It's a great idea to mark Flexbox with this bug. But I disagree with downgrading Flexbox as a whole as "partially supported". The bug only affects tables that have display: flex on the <tr>s.

Layout out HTML tables using Flexbox is not a core usecase.

Let's include the footnote, but leave the level of support.

@aardrian
Copy link
Contributor Author

It's a great idea to mark Flexbox with this bug. But I disagree with downgrading Flexbox as a whole as "partially supported". The bug only affects tables that have display: flex on the s.

I feel "only" is doing a lot of work there considering I found this because of a banking client using it this way (and another client considering it until I flagged it).

Given my small set of clients, I can only assume (perhaps poorly) it was more pervasive in non-public data table uses (where lots of financial data appears).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants