fix(sw-350): cleanups and PR comment fixes
This commit is contained in:
@@ -38,15 +38,16 @@ export default function DatePickerForm({
|
||||
|
||||
function close() {
|
||||
setIsOpen(false)
|
||||
onToggleOpen && onToggleOpen(false)
|
||||
if (onToggleOpen) {
|
||||
onToggleOpen(false)
|
||||
}
|
||||
}
|
||||
|
||||
function handleOnClick() {
|
||||
setIsOpen((prevIsOpen) => {
|
||||
const newState = !prevIsOpen
|
||||
onToggleOpen && onToggleOpen(newState)
|
||||
return newState
|
||||
})
|
||||
setIsOpen((prevIsOpen) => !prevIsOpen)
|
||||
if (onToggleOpen) {
|
||||
onToggleOpen(!isOpen)
|
||||
}
|
||||
}
|
||||
|
||||
function handleSelectDate(selected: Date) {
|
||||
@@ -64,13 +65,13 @@ export default function DatePickerForm({
|
||||
from: selectedDate.from,
|
||||
to: toDate.format("YYYY-MM-DD"),
|
||||
})
|
||||
setIsSelectingFrom(true)
|
||||
} else {
|
||||
setValue(name, {
|
||||
from: toDate.format("YYYY-MM-DD"),
|
||||
to: selectedDate.from,
|
||||
})
|
||||
}
|
||||
setIsSelectingFrom(true)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -79,7 +80,9 @@ export default function DatePickerForm({
|
||||
const target = evt.target as HTMLElement
|
||||
if (ref.current && target && !ref.current.contains(target)) {
|
||||
setIsOpen(false)
|
||||
onToggleOpen && onToggleOpen(false)
|
||||
if (onToggleOpen) {
|
||||
onToggleOpen(false)
|
||||
}
|
||||
}
|
||||
}
|
||||
document.body.addEventListener("click", handleClickOutside)
|
||||
@@ -91,7 +94,9 @@ export default function DatePickerForm({
|
||||
const selectedFromDate = dt(selectedDate.from)
|
||||
.locale(lang)
|
||||
.format("ddd D MMM")
|
||||
const selectedToDate = dt(selectedDate.to).locale(lang).format("ddd D MMM")
|
||||
const selectedToDate = !!selectedDate.to
|
||||
? dt(selectedDate.to).locale(lang).format("ddd D MMM")
|
||||
: ""
|
||||
|
||||
return (
|
||||
<div className={styles.container} data-isopen={isOpen} ref={ref}>
|
||||
|
||||
Reference in New Issue
Block a user