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

Editorial: maybe blue tables? #193

Closed
wants to merge 1 commit into from
Closed

Editorial: maybe blue tables? #193

wants to merge 1 commit into from

Conversation

spectranaut
Copy link
Contributor

@spectranaut spectranaut commented Sep 13, 2023

whaddya think @scottaohara @jnurthen @pkra ?

Trying to get rid of custom CSS - this uses respec CSS for def tables


Preview | Diff

@spectranaut
Copy link
Contributor Author

spectranaut commented Sep 13, 2023

oof it makes these tables awful: https://pr-preview.s3.amazonaws.com/w3c/core-aam/pull/193.html#mapping_events_selection

edit: oh actually those tables are already bad.

@scottaohara
Copy link
Member

yeh... i mean, it sort of is what it is unless we want to think about whether tables are really the right way to convey this information in general?

@adampage
Copy link
Member

Definition lists seem like a better fit for many of theses cases.

@pkra
Copy link
Member

pkra commented Oct 16, 2023

From a quick look, are the code elements to blame? With word-break set to break-word or break-all, it looks good. But perhaps that's not what we want.

@pkra
Copy link
Member

pkra commented Oct 16, 2023

sorry, code, not pre.

@pkra
Copy link
Member

pkra commented Oct 16, 2023

With break-all it looks better:

screenshot of spec with hacked CSS

@pkra
Copy link
Member

pkra commented May 22, 2024

@spectranaut this has merge conflicts. Can we close this and revisit it some other time?

@spectranaut
Copy link
Contributor Author

yes @pkra !

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.

None yet

4 participants