-
Notifications
You must be signed in to change notification settings - Fork 99
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(Breadcrumbs): add readme #995
Conversation
Preview is ready. |
e3ef64e
to
18e85f6
Compare
18e85f6
to
11c1207
Compare
src/components/Breadcrumbs/README.md
Outdated
import {Breadcrumbs} from '@gravity-ui/uikit'; | ||
``` | ||
|
||
`Breadcrumbs` can collapse breadcrumbs that cause overflow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need a little bit more suitable description. Smth like this
|
||
`Breadcrumbs` can collapse breadcrumbs that cause overflow. | ||
|
||
## Appearance |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
U can write about lastDisplayedItemsCount
and firstDisplayedItemsCount
either
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it extra info, I update description, if it not enough I can add info about it.
|
||
<!--/GITHUB_BLOCK--> | ||
|
||
### Custom title |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Seems like a part of
BreadcrumbsItem
item menu, but not the part of the Appearance. - I think BreadcrumbsItem could have it own props table
1e1a0e3
to
c09d5bc
Compare
c09d5bc
to
07734a7
Compare
No description provided.