From 6f9c38f65ef7bb685c56844632785df15839add8 Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Thu, 10 Dec 2020 00:38:24 +0200 Subject: [PATCH] carousel: switch to `if/else` (#32395) There are only two conditions, so `if/else` is shorter and has the same effect. --- js/src/carousel.js | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/js/src/carousel.js b/js/src/carousel.js index 299aadedaa..b5923e34a6 100644 --- a/js/src/carousel.js +++ b/js/src/carousel.js @@ -340,16 +340,12 @@ class Carousel extends BaseComponent { return } - switch (event.key) { - case ARROW_LEFT_KEY: - event.preventDefault() - this.prev() - break - case ARROW_RIGHT_KEY: - event.preventDefault() - this.next() - break - default: + if (event.key === ARROW_LEFT_KEY) { + event.preventDefault() + this.prev() + } else if (event.key === ARROW_RIGHT_KEY) { + event.preventDefault() + this.next() } }