Skip to content

Commit

Permalink
Merge e6933f1 into 25dc62b
Browse files Browse the repository at this point in the history
  • Loading branch information
Haprog committed Mar 9, 2020
2 parents 25dc62b + e6933f1 commit 7b5c668
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .travis.yml
Expand Up @@ -8,7 +8,7 @@ cache:
- node_modules

addons:
firefox: "66.0"
firefox: latest
chrome: stable

jobs:
Expand All @@ -22,12 +22,12 @@ jobs:
- if: type = pull_request
env: POLYMER=2
addons:
firefox: "66.0"
firefox: latest
chrome: stable
- if: type = pull_request
env: POLYMER=3
addons:
firefox: "66.0"
firefox: latest
chrome: stable
- if: type = cron
env: TEST_SUITE=unit_tests POLYMER=2
Expand Down
12 changes: 11 additions & 1 deletion src/vaadin-radio-button.html
Expand Up @@ -225,6 +225,10 @@
this.removeAttribute('active');

if (!this.checked && !this.disabled) {
// If you change this block, please test manually that radio-button and
// radio-group still works ok on iOS 12/13 and up as it may cause
// an issue that is not possible to test programmatically.
// See: https://github.com/vaadin/vaadin-radio-button/issues/140
this.click();
}
});
Expand All @@ -251,7 +255,13 @@
* @protected
*/
click() {
this.shadowRoot.querySelector('input').click();
// If you change this block, please test manually that radio-button and
// radio-group still works ok on iOS 12/13 and up as it may cause
// an issue that is not possible to test programmatically.
// See: https://github.com/vaadin/vaadin-radio-button/issues/140
if (!this.disabled) {
this.shadowRoot.querySelector('input').dispatchEvent(new MouseEvent('click'));
}
}

/** @protected */
Expand Down

0 comments on commit 7b5c668

Please sign in to comment.