Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[CSS] Improve Word Wrap in Parameter tables #45

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

Merged
Klustre merged 4 commits into master from fixup-arg-table-css
Jan 23, 2025
Merged

Conversation

@jthomascolombo
Copy link
Contributor

@jthomascolombo jthomascolombo commented Jan 22, 2025
edited
Loading

This is a CSS-only change to how wrapping is handled in table cells containing code. Prior to this change, many method parameter tables were wrapping poorly, with single characters broken out to their own lines for text formatted as code. Setting a nowrap override using the td code selector prevents the code-formatted function parameters from being broken into separate lines.

Testing:

Re-reviewed tables to ensure the wrapping remained consistent and correct.

Klustre reacted with thumbs up emoji
Copy link
Member

@Klustre Klustre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice detail 👍 Can you unnest the td selector and use :last-child to improve browser support? Nesting and :nth-last-child are part of the 2023 baseline so most users should have a supported browser, but the suggested alternatives have broader support and should have the same result.

Copy link
Contributor Author

Thanks, @Klustre! Those are good suggestions—I'd realized last-child might be a better fit shortly after I made the PR. Both issues are now addressed in commit e815c0d

Copy link
Contributor Author

@zlovatt suggested using the td code selector which solves the wrapping problem in a much more targeted way. Thank you, Zack!

Copy link
Member

@Klustre Klustre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, apart from the nesting within :root. The selectors should be top-level.

Copy link
Contributor Author

@Klustre Sounds good, the new selector has been moved to the top-level in 9139b91.

@Klustre Klustre merged commit d8aaf3a into master Jan 23, 2025
@jthomascolombo jthomascolombo deleted the fixup-arg-table-css branch January 23, 2025 16:57
Copy link
Member

zlovatt commented Jan 23, 2025

For what it's worth:

More info in the docsforadobe.dev repo readme

Klustre reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Klustre Klustre Klustre approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /