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

elements macro namespace #396

Closed
wants to merge 1 commit into from
Closed

Conversation

feedmypixel
Copy link
Contributor

A simple link macro that is passed into a form hint. Usage can be seen in #395 here

screen shot 2017-07-31 at 15 03 04

@ztolley ztolley temporarily deployed to datahub-beta2-pr-396 July 31, 2017 14:03 Inactive
# Render anchor markup
# @param {string} props.href - Anchor href
# @param {string} props.text - Anchor text
# @param {string} [props.target=_self] - Anchor target
Copy link
Contributor

Choose a reason for hiding this comment

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

Could the target be left empty by default and not add the attribute if not present?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This macro is going to be dropped after discussion with @tyom - closing PR

@teneightfive teneightfive deleted the feature/element-macros branch August 14, 2017 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants