Skip to content
This repository has been archived by the owner on Jan 24, 2025. It is now read-only.

Updates the PropsTable to render '[Empty String]' instead of nothing. #427

Merged
merged 2 commits into from
Oct 27, 2018

Conversation

Jared-Dev
Copy link
Contributor

Description

Adds [Empty String] or [No Default] to the "Default" column depending on the prop.

Screenshots

Before

If it wasn't for the prop name...which prop had no default and which one is an empty string?
image

After

image

@Jared-Dev Jared-Dev requested a review from pedronauck October 27, 2018 01:24
@pedronauck pedronauck merged commit 4669e34 into doczjs:master Oct 27, 2018
pedronauck pushed a commit that referenced this pull request Oct 27, 2018
* Update the PropsTable to conditionally include the description column.

* Adds a description to the Alert component in the
basic examples to demonstrate it being added
* The Button component in the basic example has no
descriptions to demonstrate it being excluded

* Remove a data- attribute that was used for testing

* feat(docz): render [Empty String] on PropsTable (#427)

* Updates the PropsTable to render '[Empty String]' instead of nothing.
* Add the code to print [No Default]

* fix(docz-theme-default): prop table header background was overflowing (#421)
pedronauck pushed a commit that referenced this pull request Oct 30, 2018
* Updates the PropsTable to render '[Empty String]' instead of nothing.
* Add the code to print [No Default]
pedronauck pushed a commit that referenced this pull request Oct 30, 2018
* Updates the PropsTable to render '[Empty String]' instead of nothing.
* Add the code to print [No Default]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants