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

feat: clearInput API #151

Merged
merged 3 commits into from
Aug 3, 2017
Merged

Conversation

criticalbh
Copy link
Contributor

@criticalbh criticalbh commented Aug 2, 2017

Closes #133

@@ -31,6 +31,7 @@ import {
mousedown,
mouseup,
focus,
clear
Copy link
Owner

Choose a reason for hiding this comment

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

Trailing comma

src/types.ts Outdated
@@ -136,6 +136,10 @@ export type Command =
type: 'focus'
selector: string
}
| {
type: 'clear',
Copy link
Owner

Choose a reason for hiding this comment

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

No comma in type defs

CHANGELOG.md Outdated
@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

### Added
- `placeholder()` API method [#000](https://github.com/graphcool/chromeless/pull/000) @contributor
- `clear()` API method
Copy link
Owner

Choose a reason for hiding this comment

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

I'd be in favour of calling this clearInput to be more explicit.

Copy link
Collaborator

@adieuadieu adieuadieu left a comment

Choose a reason for hiding this comment

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

Thank you @criticalbh!

@adieuadieu adieuadieu added this to the v1.2 milestone Aug 3, 2017
@adieuadieu adieuadieu merged commit 34ee149 into schickling:master Aug 3, 2017
@adieuadieu adieuadieu changed the title feat: clear API feat: clearInput API Aug 3, 2017
@criticalbh criticalbh deleted the feature/clear branch August 3, 2017 19:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants