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

Docs: quick fixes for HTML #36395

Merged
merged 7 commits into from May 20, 2022
Merged

Conversation

louismaximepiton
Copy link
Member

@louismaximepiton louismaximepiton commented May 19, 2022

Here are the issues concerning the CSS part, we encountered with @julien-deramond.

The commits can be splited if needed or wanted or even be in another PR. I'm open to every comment that could help fix these issues.

Issues tackled by this PR:

New helper with (47a4d3d)

Using the new helper in doc and examples whenever it's feasible.

tabindex="-1" with (73a86e8)

Remove the access, for keyboard users, to the input that is unused and here to avoid a Safari bug (#36176 (comment))

Scrollspy examples with (651d66e)

Change scrollspy code examples to be closer to the real implementation

Re-order js tables in doc with (6c9b30a)

Re-ordering alphabetically js tables in doc to be more comprehensive and user friendly (except hide with hidden to keep logic)

Parcel with (f7b83ca)

Fix some doc inconsistencies in parcel.md.

Align center with (212ffe0)

Align center almost all the texts in examples in Category Layout. I found it more visual and followed the CSS-grid example. If these are approved, it might be good to backport it to flex utilities, etc...

Re-order helpers with (eb4dc49)

Re-ordering alphabetically helpers. if these changes are approved, might be great to bring it back to other main categories such as Layout, Forms, etc...

@louismaximepiton louismaximepiton marked this pull request as ready for review May 20, 2022 14:01
Copy link
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

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

Thank you for the detailed commits and PR!

v5.2.0-stable automation moved this from In progress to Reviewer approved May 20, 2022
@mdo mdo merged commit 8959bf3 into twbs:main May 20, 2022
v5.2.0-stable automation moved this from Reviewer approved to Done May 20, 2022
@louismaximepiton louismaximepiton deleted the main-lmp-docs-fixes branch September 2, 2022 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
v5.2.0-stable
  
Done
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants