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

Documentation - Add tables to explain API parameters #638

Closed
paulinea opened this issue Sep 21, 2023 · 2 comments · Fixed by #658
Closed

Documentation - Add tables to explain API parameters #638

paulinea opened this issue Sep 21, 2023 · 2 comments · Fixed by #658
Labels
📖 documentation Improvements or additions to documentation 👌 OK PO

Comments

@paulinea
Copy link
Member

Add a table in markdown file can be done as follow:

Parameters | Descriptions
-- | --
`message: String` | Text to be displayed into the banner
`firstButton: OdsBannerButton` | Principal button to be displayed in the banner
`modifier: Modifier` | `Modifier` to be applied to the banner
`image: OdsBannerImage?` | Image to be displayed in the banner in a circle shape
`secondButton: OdsBannerButton?` | Secondary button to be displayed in the banner
{:.table}

Where {:.table} allows to apply Boosted table style.

@paulinea paulinea added the 📖 documentation Improvements or additions to documentation label Sep 21, 2023
paulinea added a commit that referenced this issue Oct 5, 2023
paulinea added a commit that referenced this issue Oct 5, 2023
paulinea added a commit that referenced this issue Oct 6, 2023
paulinea added a commit that referenced this issue Oct 6, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 9, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
paulinea added a commit that referenced this issue Oct 13, 2023
@paulinea paulinea assigned B3nz01d and unassigned paulinea Oct 13, 2023
@B3nz01d
Copy link
Collaborator

B3nz01d commented Oct 26, 2023

@paulinea I cannot test this yet right?
I do not see tables in the current exposed doc

@paulinea
Copy link
Member Author

@B3nz01d I sent you the link to test by Teams several days ago: https://deploy-preview-658--ods-android.netlify.app/components/buttons_docs

@B3nz01d B3nz01d removed their assignment Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📖 documentation Improvements or additions to documentation 👌 OK PO
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants