Skip to content

Update utilities API docs#32049

Merged
XhmikosR merged 4 commits into
mainfrom
utils-api-edits
Nov 5, 2020
Merged

Update utilities API docs#32049
XhmikosR merged 4 commits into
mainfrom
utils-api-edits

Conversation

@mdo

@mdo mdo commented Nov 2, 2020

Copy link
Copy Markdown
Member

Changes so far:

  • Reformats list of utilities options into a responsive table.
  • Did some copyediting on body text and headings.
  • Cleaned up the formatting of our outputted CSS so it's not as lengthy of a page.

Still to do:

/cc @MartijnCuppens @XhmikosR

Preview: https://deploy-preview-32049--twbs-bootstrap.netlify.app/docs/5.0/utilities/api/

Closes #31328 by superseding it

@XhmikosR XhmikosR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM but I'd like a look from @MartijnCuppens

@MartijnCuppens

Copy link
Copy Markdown
Member

Looks good so far!

@mdo mdo force-pushed the utils-api-edits branch from 9cbeedd to fd08f46 Compare November 3, 2020 18:45
@mdo mdo marked this pull request as ready for review November 3, 2020 18:45
@mdo

mdo commented Nov 3, 2020

Copy link
Copy Markdown
Member Author

Alrighty this is ready to go I think! Tested all the new snippets I added under the Using the API section—everything should be good now.

mdo and others added 3 commits November 4, 2020 08:40
Moves the remote utilities section to the using the api section and updates the code snippet to work. Previous one removed all utilities.
@XhmikosR XhmikosR changed the title WIP: Updated utilities API docs Updated utilities API docs Nov 4, 2020
@XhmikosR XhmikosR changed the title Updated utilities API docs Update utilities API docs Nov 4, 2020
@XhmikosR XhmikosR merged commit 0766a09 into main Nov 5, 2020
@XhmikosR XhmikosR deleted the utils-api-edits branch November 5, 2020 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants