Skip to content

Commit

Permalink
Merge pull request #42 from ionic-team/master
Browse files Browse the repository at this point in the history
pull `master`
  • Loading branch information
abennouna authored Jan 19, 2019
2 parents 11e8db9 + 7832be3 commit f0ed0f9
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 9 deletions.
10 changes: 5 additions & 5 deletions angular/test/test-app/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"ionicons": "4.5.1"
},
"devDependencies": {
"@stencil/core": "0.16.3",
"@stencil/core": "0.17.0",
"@stencil/sass": "0.1.1",
"@stencil/utils": "latest",
"@types/jest": "^23.3.1",
Expand Down
3 changes: 1 addition & 2 deletions core/src/components/button/button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ export class Button implements ComponentInterface {
@Listen('click')
onClick(ev: Event) {
if (this.type === 'button') {
return openURL(this.win, this.href, ev, this.routerDirection);
openURL(this.win, this.href, ev, this.routerDirection);

} else if (hasShadowDom(this.el)) {
// this button wants to specifically submit a form
Expand All @@ -122,7 +122,6 @@ export class Button implements ComponentInterface {
fakeButton.remove();
}
}
return Promise.resolve(false);
}

private onFocus = () => {
Expand Down
2 changes: 1 addition & 1 deletion core/src/components/toast/toast.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ export class Toast implements ComponentInterface, OverlayInterface {
<div class="toast-message">{this.message}</div>
}
{this.showCloseButton &&
<ion-button fill="clear" class="toast-button ion-activatable" onClick={() => this.dismiss(undefined, 'cancel')}>
<ion-button fill="clear" class="toast-button" onClick={() => this.dismiss(undefined, 'cancel')}>
{this.closeButtonText || 'Close'}
</ion-button>
}
Expand Down

0 comments on commit f0ed0f9

Please sign in to comment.