-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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] Divider, DropDownMenu, FlatButton, FloatingActionButton, FontIcon - Add title & description to examples #2955
[Docs] Divider, DropDownMenu, FlatButton, FloatingActionButton, FontIcon - Add title & description to examples #2955
Conversation
Please check FontIcon in particular, as I wasn't totally sure of the reason the simple example only required a classname, while the other has the icon name as a child - or whether calling these Custom and Public was appropriate. |
const descriptions = { | ||
simple: '`FlatButton` with default color, `primary`, `secondary` and and `disabled` props applied.', | ||
complex: 'The first example uses an `input` as a child component, ' + | ||
'the next has next has an [SVG Icon](http://localhost:3000/#/components/svg-icon), with the label positioned after.' + |
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.
We should remove http://localhost:3000
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.
Ach, sorry.
That's a good point. Those classnames are declared outside of the examples, they are private to the documentation. I have seen many issues related to this. |
simple: '`FlatButton` with default color, `primary`, `secondary` and and `disabled` props applied.', | ||
complex: 'The first example uses an `input` as a child component, ' + | ||
'the next has next has an [SVG Icon](/#/components/svg-icon), with the label positioned after. ' + | ||
'The final example uses a [FontIcon](/#/components/font-icon, and is wrapped in an anchor tag.', |
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.
font-icon**)**, and
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.
Oh my!
Fixed up and squashed onto the FlatButton commit. |
[Docs] Divider, DropDownMenu, FlatButton, FloatingActionButton, FontIcon - Add title & description to examples
@mbrookes Thanks |
For #2927.