mirror_css/bootstrap
1
0
Fork 0
mirror of https://github.com/twbs/bootstrap.git synced 2025-08-13 17:14:04 +02:00
Code Issues Releases Wiki Activity

Carousel: omit redundant checks as we are always transforming the right values

Browse Source
This commit is contained in:
GeoSot
2022-03-10 00:38:17 +02:00
committed by XhmikosR
parent 28f150d720
commit 88da704eed
2 changed files with 0 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
js/src/carousel.js
View File

@@ -389,10 +389,6 @@ class Carousel extends BaseComponent {
} }
_directionToOrder(direction) { _directionToOrder(direction) {
if (![DIRECTION_RIGHT, DIRECTION_LEFT].includes(direction)) {
return direction
}
if (isRTL()) { if (isRTL()) {
return direction === DIRECTION_LEFT ? ORDER_PREV : ORDER_NEXT return direction === DIRECTION_LEFT ? ORDER_PREV : ORDER_NEXT
} }
@@ -401,10 +397,6 @@ class Carousel extends BaseComponent {
} }
_orderToDirection(order) { _orderToDirection(order) {
if (![ORDER_NEXT, ORDER_PREV].includes(order)) {
return order
}
if (isRTL()) { if (isRTL()) {
return order === ORDER_PREV ? DIRECTION_LEFT : DIRECTION_RIGHT return order === ORDER_PREV ? DIRECTION_LEFT : DIRECTION_RIGHT
} }

4
js/tests/unit/carousel.spec.js
View File

@@ -1200,9 +1200,7 @@ describe('Carousel', () => {
const carousel = new Carousel(carouselEl, {}) const carousel = new Carousel(carouselEl, {})
expect(carousel._directionToOrder('left')).toEqual('next') expect(carousel._directionToOrder('left')).toEqual('next')
expect(carousel._directionToOrder('prev')).toEqual('prev')
expect(carousel._directionToOrder('right')).toEqual('prev') expect(carousel._directionToOrder('right')).toEqual('prev')
expect(carousel._directionToOrder('next')).toEqual('next')
expect(carousel._orderToDirection('next')).toEqual('left') expect(carousel._orderToDirection('next')).toEqual('left')
expect(carousel._orderToDirection('prev')).toEqual('right') expect(carousel._orderToDirection('prev')).toEqual('right')
@@ -1217,9 +1215,7 @@ describe('Carousel', () => {
expect(isRTL()).toBeTrue() expect(isRTL()).toBeTrue()
expect(carousel._directionToOrder('left')).toEqual('prev') expect(carousel._directionToOrder('left')).toEqual('prev')
expect(carousel._directionToOrder('prev')).toEqual('prev')
expect(carousel._directionToOrder('right')).toEqual('next') expect(carousel._directionToOrder('right')).toEqual('next')
expect(carousel._directionToOrder('next')).toEqual('next')
expect(carousel._orderToDirection('next')).toEqual('right') expect(carousel._orderToDirection('next')).toEqual('right')
expect(carousel._orderToDirection('prev')).toEqual('left') expect(carousel._orderToDirection('prev')).toEqual('left')