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

fix(Input): fix placeholder style #415

Merged
merged 3 commits into from
Feb 19, 2019
Merged

fix(Input): fix placeholder style #415

merged 3 commits into from
Feb 19, 2019

Conversation

sun-tea
Copy link
Contributor

@sun-tea sun-tea commented Feb 19, 2019

  • Feature
  • Fix
  • Enhancement

Brief

Use placeholder pseudo-element instead of deprecated method from polished lib.

Description

Polished dependency deprecated its placeholder() method, so we need to remove it from our lib.
Stardust is broken because of that in captain-client (weirdly it broke on update of polished dependency in captain-client, I would have thought that inner dependency in Stardust would not be impacted?). Either way this PR is fixing it.

Requirements :

⚠️ Requires running yarn command.

Todo - Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.

@spartDev spartDev merged commit d3abc9b into master Feb 19, 2019
@spartDev spartDev deleted the fix-placeholder branch February 19, 2019 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants