-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Paper] Add elevation boundaries #6817
[Paper] Add elevation boundaries #6817
Conversation
oliviertassinari
commented
May 8, 2017
- PR has tests / docs demo, and is linted.
- Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
- Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).
src/Hidden/Hidden.js
Outdated
@@ -1,9 +1,61 @@ | |||
// @flow | |||
import React, { Element } from 'react'; | |||
import HiddenJs from './HiddenJs'; | |||
import type { HiddenProps } from './types'; |
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.
Duplicating the definitions as react-docgen isn't smart enough to resolve imports.
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 am very strongly against duplicating anything at the moment. I have an issue raised and have been thinking about the solution. Duplication is a last resort and I don't think we are there yet. I will raise an issue though so we are tracking it.
@rosskevin Right now it's already duplicated between the documentation |
Please see #6818. We don't need to resort to duplication yet. |
Removed every changes related to Hidden.