Skip to content

Commit

Permalink
chore: linting
Browse files Browse the repository at this point in the history
  • Loading branch information
bashleigh committed Mar 6, 2024
1 parent c67a06d commit dd5abdd
Show file tree
Hide file tree
Showing 16 changed files with 145 additions and 157 deletions.
8 changes: 4 additions & 4 deletions src/components/card/__tests__/card-components.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ describe('handleToggleContextMenu', () => {
stopPropagation: jest.fn(),
}

curried((event as unknown) as MouseEvent)
curried(event as unknown as MouseEvent)

expect(mockSetValue).toHaveBeenCalledTimes(1)
expect(mockSetValue).toHaveBeenCalledWith(true)
Expand All @@ -113,7 +113,7 @@ describe('handleToggleMainMobileOpen', () => {
stopPropagation: jest.fn(),
}

curried((event as unknown) as MouseEvent)
curried(event as unknown as MouseEvent)

expect(mockSetValue).toHaveBeenCalledTimes(1)
expect(mockSetValue).toHaveBeenCalledWith(true)
Expand All @@ -130,7 +130,7 @@ describe('handleToggleListMobileOpen', () => {
stopPropagation: jest.fn(),
}

curried((event as unknown) as MouseEvent)
curried(event as unknown as MouseEvent)

expect(mockSetValue).toHaveBeenCalledTimes(1)
expect(mockSetValue).toHaveBeenCalledWith(true)
Expand All @@ -149,7 +149,7 @@ describe('handleToggleBothMobileOpen', () => {
stopPropagation: jest.fn(),
}

curried((event as unknown) as MouseEvent)
curried(event as unknown as MouseEvent)

expect(mockSetValue).toHaveBeenCalledTimes(1)
expect(mockSetValue).toHaveBeenCalledWith(true)
Expand Down
6 changes: 3 additions & 3 deletions src/components/file-input/__tests__/file-input.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe('handleFileView', () => {

const curried = handleFileView(onFileView, fileUrl)

curried((event as unknown) as MouseEvent<HTMLSpanElement>)
curried(event as unknown as MouseEvent<HTMLSpanElement>)

expect(onFileView).toHaveBeenCalledWith(fileUrl)
expect(event.stopPropagation).toHaveBeenCalledTimes(1)
Expand All @@ -58,7 +58,7 @@ describe('handleFileClear', () => {

const curried = handleFileClear(setFileName)

curried((event as unknown) as MouseEvent<HTMLSpanElement>)
curried(event as unknown as MouseEvent<HTMLSpanElement>)

expect(setFileName).toHaveBeenCalledWith('')
expect(event.stopPropagation).toHaveBeenCalledTimes(1)
Expand All @@ -79,7 +79,7 @@ describe('handleFileChange', () => {

const curried = handleFileChange(setFileName, fileName, onFileUpload)

const reader = curried((event as unknown) as ChangeEvent<HTMLInputElement>)
const reader = curried(event as unknown as ChangeEvent<HTMLInputElement>)

expect(reader instanceof FileReader).toBe(true)
})
Expand Down
90 changes: 45 additions & 45 deletions src/components/file-input/file-input.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -53,60 +53,60 @@ export interface ImageUploadModel {
Url: string
}

export const handleFileChange = (
setFileName: Dispatch<SetStateAction<string>>,
fileName: string,
onFileUpload?: (uploadImageModel: CreateImageUploadModel) => Promise<string | ImageUploadModel>,
) => (event: ChangeEvent<HTMLInputElement>) => {
if (event.target && event.target.files && event.target.files[0]) {
const file = event.target.files[0]

const fileUrl = URL.createObjectURL(file)

if (typeof fileUrl === 'string') {
setFileName(fileUrl)
}
export const handleFileChange =
(
setFileName: Dispatch<SetStateAction<string>>,
fileName: string,
onFileUpload?: (uploadImageModel: CreateImageUploadModel) => Promise<string | ImageUploadModel>,
) =>
(event: ChangeEvent<HTMLInputElement>) => {
if (event.target && event.target.files && event.target.files[0]) {
const file = event.target.files[0]

const fileUrl = URL.createObjectURL(file)

if (typeof fileUrl === 'string') {
setFileName(fileUrl)
}

const reader = new FileReader()
reader.readAsDataURL(file)
reader.onload = async () => {
const base64 = reader.result
const reader = new FileReader()
reader.readAsDataURL(file)
reader.onload = async () => {
const base64 = reader.result

if (onFileUpload && typeof base64 === 'string') {
const uploaded = await onFileUpload({
imageData: base64,
name: `${fileName ? fileName : file.name}`,
})
if (onFileUpload && typeof base64 === 'string') {
const uploaded = await onFileUpload({
imageData: base64,
name: `${fileName ? fileName : file.name}`,
})

if (uploaded && (uploaded as ImageUploadModel).Url) {
setFileName((uploaded as ImageUploadModel).Url)
if (uploaded && (uploaded as ImageUploadModel).Url) {
setFileName((uploaded as ImageUploadModel).Url)
}
}
}
}
reader.onerror = (error) => {
console.error(`file upload error: ${error}`)
}
reader.onerror = (error) => {
console.error(`file upload error: ${error}`)
}

return reader
return reader
}
}
}

export const handleFileClear = (setFileName: Dispatch<SetStateAction<string>>) => (
event: MouseEvent<HTMLSpanElement>,
) => {
event.stopPropagation()
event.preventDefault()
export const handleFileClear =
(setFileName: Dispatch<SetStateAction<string>>) => (event: MouseEvent<HTMLSpanElement>) => {
event.stopPropagation()
event.preventDefault()

setFileName('')
}
setFileName('')
}

export const handleFileView = (onFileView: (fileUrl: string) => void, fileUrl: string) => (
event: MouseEvent<HTMLSpanElement>,
) => {
event.stopPropagation()
event.preventDefault()
onFileView(fileUrl)
}
export const handleFileView =
(onFileView: (fileUrl: string) => void, fileUrl: string) => (event: MouseEvent<HTMLSpanElement>) => {
event.stopPropagation()
event.preventDefault()
onFileView(fileUrl)
}

export const FilePreviewImage: FC<FilePreviewImageProps> = ({ src }) => {
if (!src) return <PlaceholderImage placeholder="placeholderSmall" size={120} fillAvailable />
Expand Down Expand Up @@ -152,7 +152,7 @@ export const FileInput: FileInputWrapped = forwardRef(
id={inputId}
{...rest}
defaultValue={defaultValue}
ref={(ref as unknown) as LegacyRef<HTMLInputElement>}
ref={ref as unknown as LegacyRef<HTMLInputElement>}
/>
{fileUrl ? (
<ElFileInputIconContainer>
Expand Down
17 changes: 8 additions & 9 deletions src/components/form-layout/form-layout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,14 @@ export const FormSectionDivider: FC<FormLayoutProps> = ({ children, ...rest }) =
return <ElFormSectionDivider {...rest}>{children}</ElFormSectionDivider>
}

export const InputWrap: React.ForwardRefExoticComponent<
FormLayoutProps & React.RefAttributes<HTMLDivElement>
> = forwardRef(({ children, ...rest }, ref) => {
return (
<ElInputWrap {...rest} ref={ref as LegacyRef<HTMLDivElement>}>
{children}
</ElInputWrap>
)
})
export const InputWrap: React.ForwardRefExoticComponent<FormLayoutProps & React.RefAttributes<HTMLDivElement>> =
forwardRef(({ children, ...rest }, ref) => {
return (
<ElInputWrap {...rest} ref={ref as LegacyRef<HTMLDivElement>}>
{children}
</ElInputWrap>
)
})

export const InputWrapSmall: FC<FormLayoutProps> = ({ children, ...rest }) => {
return <ElInputWrapSmall {...rest}>{children}</ElInputWrapSmall>
Expand Down
2 changes: 1 addition & 1 deletion src/components/input/input.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export const Input: InputWrapped = forwardRef(
<ElInput
className={cx(hasError && elHasInputError)}
{...rest}
ref={(ref as unknown) as LegacyRef<HTMLInputElement>}
ref={ref as unknown as LegacyRef<HTMLInputElement>}
/>
)
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,10 @@ describe('clickEventHandler', () => {
it('should handle a click event', () => {
const setActive = jest.fn()
const onClick = jest.fn()
const event = ({
const event = {
preventDefault: jest.fn(),
stopPropagation: jest.fn(),
} as unknown) as MouseEvent<HTMLAnchorElement | HTMLDivElement>
} as unknown as MouseEvent<HTMLAnchorElement | HTMLDivElement>

const curried = clickEventHandler(setActive, onClick)

Expand Down
18 changes: 9 additions & 9 deletions src/components/mobile-controls/mobile-controls.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,18 +17,18 @@ export interface MobileControlsProps extends HTMLAttributes<HTMLDivElement> {
isVisible?: boolean
}

export const clickEventHandler = (setActive: Dispatch<SetStateAction<boolean>>, onClick?: () => void) => (
event: MouseEvent<HTMLAnchorElement | HTMLDivElement>,
) => {
event?.preventDefault()
event?.stopPropagation()
export const clickEventHandler =
(setActive: Dispatch<SetStateAction<boolean>>, onClick?: () => void) =>
(event: MouseEvent<HTMLAnchorElement | HTMLDivElement>) => {
event?.preventDefault()
event?.stopPropagation()

setActive((active) => !active)
setActive((active) => !active)

if (onClick) {
onClick()
if (onClick) {
onClick()
}
}
}

export const MobileControls: FC<MobileControlsProps> = ({
mobileControlItems,
Expand Down
4 changes: 2 additions & 2 deletions src/components/nav/__tests__/nav-responsive.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,10 @@ describe('handleToggleLogo', () => {
describe('clickNavEventHandler', () => {
it('should handle a click event', () => {
const setActive = jest.fn()
const event = ({
const event = {
preventDefault: jest.fn(),
stopPropagation: jest.fn(),
} as unknown) as MouseEvent<HTMLAnchorElement | HTMLDivElement>
} as unknown as MouseEvent<HTMLAnchorElement | HTMLDivElement>

const curried = clickNavEventHandler(setActive)

Expand Down
43 changes: 18 additions & 25 deletions src/components/pagination/pagination.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,35 +34,28 @@ export interface PaginationInputProps extends InputHTMLAttributes<HTMLInputEleme

export interface PaginationButtonProps extends HTMLAttributes<HTMLButtonElement> {}

export const handlePageChange = (
nextPage: number | null,
callback: (page: number) => void,
setInputValue: Dispatch<SetStateAction<string>>,
) => () => {
if (nextPage) {
setInputValue(String(nextPage))
callback(nextPage)
export const handlePageChange =
(nextPage: number | null, callback: (page: number) => void, setInputValue: Dispatch<SetStateAction<string>>) =>
() => {
if (nextPage) {
setInputValue(String(nextPage))
callback(nextPage)
}
}
}

export const handlePageInputChange = (
numberPages: number,
currentPage: number,
inputValue: string,
callback: (page: number) => void,
) => () => {
const nextPage = Number(inputValue)
if (nextPage && nextPage <= numberPages && nextPage !== currentPage) {
callback(nextPage)
export const handlePageInputChange =
(numberPages: number, currentPage: number, inputValue: string, callback: (page: number) => void) => () => {
const nextPage = Number(inputValue)
if (nextPage && nextPage <= numberPages && nextPage !== currentPage) {
callback(nextPage)
}
}
}

export const handlePageInput = (setInputValue: Dispatch<SetStateAction<string>>) => (
event: ChangeEvent<HTMLInputElement>,
) => {
const nextPage = event.target.value
setInputValue(nextPage)
}
export const handlePageInput =
(setInputValue: Dispatch<SetStateAction<string>>) => (event: ChangeEvent<HTMLInputElement>) => {
const nextPage = event.target.value
setInputValue(nextPage)
}

export const PaginationWrap: FC<PaginationWrapProps> = ({ children, ...rest }) => (
<ElPaginationWrap {...rest}>{children}</ElPaginationWrap>
Expand Down
41 changes: 18 additions & 23 deletions src/components/progress-bar/progress-bar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,22 +36,20 @@ export const ProgressBarLabel: FC<ProgressBarBaseProps> = ({ children, ...rest }
<ElProgressBarLabel {...rest}>{children}</ElProgressBarLabel>
)

export const handleSetPercentageComplete = (
setPercentageComplete: Dispatch<SetStateAction<number>>,
intervalTime: number,
) => () => {
const interval = window.setInterval(() => {
setPercentageComplete((prev) => {
if (prev < 100) {
return ++prev
}

return prev
})
}, intervalTime)

return () => window.clearInterval(interval)
}
export const handleSetPercentageComplete =
(setPercentageComplete: Dispatch<SetStateAction<number>>, intervalTime: number) => () => {
const interval = window.setInterval(() => {
setPercentageComplete((prev) => {
if (prev < 100) {
return ++prev
}

return prev
})
}, intervalTime)

return () => window.clearInterval(interval)
}

export const ProgressBarPercentage: FC<ProgressBarPercentageProps> = ({ duration, showLabel = true, ...rest }) => {
const [percentageComplete, setPercentageComplete] = useState<number>(0)
Expand All @@ -76,13 +74,10 @@ export const ProgressBarPercentage: FC<ProgressBarPercentageProps> = ({ duration
)
}

export const handleSetPercentageCompleteSteps = (
setPercentageComplete: Dispatch<SetStateAction<number>>,
currentStep: number,
numberSteps: number,
) => () => {
setPercentageComplete((currentStep / numberSteps) * 100)
}
export const handleSetPercentageCompleteSteps =
(setPercentageComplete: Dispatch<SetStateAction<number>>, currentStep: number, numberSteps: number) => () => {
setPercentageComplete((currentStep / numberSteps) * 100)
}

export const ProgressBarSteps: FC<ProgressBarStepProps> = ({ numberSteps, currentStep, showLabel = true, ...rest }) => {
const [percentageComplete, setPercentageComplete] = useState<number>((currentStep / numberSteps) * 100)
Expand Down
2 changes: 1 addition & 1 deletion src/components/select/select.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ export type SelectWrapped = React.ForwardRefExoticComponent<
export const Select: SelectWrapped = forwardRef(
({ children, ...rest }, ref: React.ForwardedRef<React.SelectHTMLAttributes<HTMLSelectElement>>) => {
return (
<ElSelect {...rest} ref={(ref as unknown) as LegacyRef<HTMLSelectElement>}>
<ElSelect {...rest} ref={ref as unknown as LegacyRef<HTMLSelectElement>}>
{children}
</ElSelect>
)
Expand Down
Loading

0 comments on commit dd5abdd

Please sign in to comment.