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: InputGrid and InputWithDecorations #1539

Merged
merged 12 commits into from
Sep 5, 2023
Merged

Conversation

gretanausedaite
Copy link
Contributor

@gretanausedaite gretanausedaite commented Aug 31, 2023

Changes

Added docs pages for InputWithDecorations, InputGrid and StatusMessage.

Testing

N/A

Docs

N/A

@gretanausedaite gretanausedaite self-assigned this Aug 31, 2023
@gretanausedaite gretanausedaite marked this pull request as ready for review September 1, 2023 12:59
@gretanausedaite gretanausedaite requested review from a team as code owners September 1, 2023 12:59
@gretanausedaite gretanausedaite requested review from mayank99 and r100-stack and removed request for a team September 1, 2023 12:59
apps/website/src/pages/docs/inputwithdecorations.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/inputwithdecorations.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/inputwithdecorations.mdx Outdated Show resolved Hide resolved
@@ -17,7 +17,7 @@ import * as AllExamples from 'examples';
<AllExamples.InputGroupToggleSwitchExample client:load />
</LiveExample>

Input group class is a container to enhance an input by adding an icon, text, or a button in front or behind the input field as a "help text".
Input group is a container to enhance an input by adding an icon, text, or a button in front or behind the input field as a "help text".
Copy link
Contributor

Choose a reason for hiding this comment

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

i think input group is only for radio/checkbox/toggle rather than regular inputs, so it might be worth mentioning

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was not prepared to update InputGroup docs because description is also wrong, just saw a small mistake in text. I can look into it more if you'd like. Or undo this change.

apps/website/src/pages/docs/inputgrid.mdx Outdated Show resolved Hide resolved
examples/InputWithDecorations.advanced.tsx Show resolved Hide resolved
examples/InputWithDecorations.main.tsx Show resolved Hide resolved
apps/website/src/pages/docs/inputgrid.mdx Show resolved Hide resolved
apps/website/src/pages/docs/statusmessage.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/statusmessage.mdx Outdated Show resolved Hide resolved
Copy link
Member

@r100-stack r100-stack left a comment

Choose a reason for hiding this comment

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

No additional comments so far

Copy link
Member

@r100-stack r100-stack left a comment

Choose a reason for hiding this comment

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

optional: Noticed that formatting of the bullet points under the InputGrid's props section looks incorrect.

image

@gretanausedaite
Copy link
Contributor Author

optional: Noticed that formatting of the bullet points under the InputGrid's props section looks incorrect.

image

This is generated automatically. Will not be fixed in this PR.

@gretanausedaite gretanausedaite added this pull request to the merge queue Sep 5, 2023
@mayank99 mayank99 mentioned this pull request Sep 5, 2023
21 tasks
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 5, 2023
@mayank99 mayank99 merged commit 816372e into dev Sep 5, 2023
@mayank99 mayank99 deleted the greta/input-docs branch September 5, 2023 14:46
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.

3 participants