Skip to content
Snippets Groups Projects
Commit 9027a683 authored by Lukas's avatar Lukas
Browse files

Fix build warnings

parent 18634386
No related branches found
No related tags found
1 merge request!530Lbudginas/#432 build warnings
Pipeline #113950 failed
...@@ -145,6 +145,7 @@ const CommentsSearch: React.FC<CommentsSearchProps> = ({ ...@@ -145,6 +145,7 @@ const CommentsSearch: React.FC<CommentsSearchProps> = ({
setCommentSearchProps(undefined); setCommentSearchProps(undefined);
setSelectedCommentSearchOperationTypes([]); setSelectedCommentSearchOperationTypes([]);
}; };
//eslint-disable-next-line react-hooks/exhaustive-deps
}, [isAccountPage, commentsSearchPermlink]); }, [isAccountPage, commentsSearchPermlink]);
const infoText = isAccountPage const infoText = isAccountPage
......
...@@ -39,8 +39,8 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -39,8 +39,8 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
const [inputFocus, setInputFocus] = useState(false); const [inputFocus, setInputFocus] = useState(false);
const [selectedResult, setSelectedResult] = useState(0); const [selectedResult, setSelectedResult] = useState(0);
const searchContainerRef = useRef(null); const searchContainerRef = useRef(null);
const inputRef = useRef<HTMLInputElement>(null); const inputRef = useRef<HTMLInputElement>(null);
const selectedResultRef = useRef<HTMLDivElement>(null); const selectedResultRef = useRef<HTMLDivElement>(null);
const [searchInputType, setSearchInputType] = useState<string>(""); const [searchInputType, setSearchInputType] = useState<string>("");
const { inputTypeData } = useInputType(searchInputType); const { inputTypeData } = useInputType(searchInputType);
const [searchTerm, setSearchTerm] = useState(""); const [searchTerm, setSearchTerm] = useState("");
...@@ -62,11 +62,11 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -62,11 +62,11 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
const isNumeric = (value: string): boolean => { const isNumeric = (value: string): boolean => {
return /^\d+$/.test(value); return /^\d+$/.test(value);
}; };
const isHash = (value: string): boolean => { const isHash = (value: string): boolean => {
return /^[a-fA-F0-9]{40}$/.test(value); return /^[a-fA-F0-9]{40}$/.test(value);
}; };
const handleInputChange = (e: React.ChangeEvent<HTMLInputElement>) => { const handleInputChange = (e: React.ChangeEvent<HTMLInputElement>) => {
setInputFocus(true); setInputFocus(true);
onChange(e.target.value); onChange(e.target.value);
...@@ -81,7 +81,7 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -81,7 +81,7 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
// Close the search when clicking outside of the container // Close the search when clicking outside of the container
useOnClickOutside(searchContainerRef, () => closeSearchBar()); useOnClickOutside(searchContainerRef, () => closeSearchBar());
const resetSearchBar = () => { const resetSearchBar = useCallback(() => {
setInputFocus(false); setInputFocus(false);
onChange(""); // Clear the input field onChange(""); // Clear the input field
setSearchInputType(""); setSearchInputType("");
...@@ -90,12 +90,12 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -90,12 +90,12 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
if (onBlur) { if (onBlur) {
onBlur({} as React.FocusEvent<HTMLInputElement>); onBlur({} as React.FocusEvent<HTMLInputElement>);
} }
}; }, [onBlur, onChange]);
const closeSearchBar = () => { const closeSearchBar = () => {
setInputFocus(false); setInputFocus(false);
}; };
// Ensure cleaning the searchbar when navigating away // Ensure cleaning the searchbar when navigating away
useEffect(() => { useEffect(() => {
const handleRouteChange = () => { const handleRouteChange = () => {
...@@ -127,10 +127,11 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -127,10 +127,11 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
if (event.key === "ArrowUp") { if (event.key === "ArrowUp") {
setSelectedResult((prev) => (prev > 0 ? prev - 1 : prev)); setSelectedResult((prev) => (prev > 0 ? prev - 1 : prev));
} }
if (event.key === "Enter") { if (event.key === "Enter") {
if (isItemSelected && linkResult) { if (isItemSelected && linkResult) {
const href = const href =
inputTypeData.input_type === "account_name" || inputTypeData.input_type === "account_name_array" inputTypeData.input_type === "account_name" ||
inputTypeData.input_type === "account_name_array"
? `/@${selectedAccount}` ? `/@${selectedAccount}`
: `/${getResultTypeHeader(inputTypeData)}/${selectedAccount}`; : `/${getResultTypeHeader(inputTypeData)}/${selectedAccount}`;
router.push(href).then(() => { router.push(href).then(() => {
...@@ -169,7 +170,8 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -169,7 +170,8 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
const handleBlur = (event: React.FocusEvent<HTMLInputElement>) => { const handleBlur = (event: React.FocusEvent<HTMLInputElement>) => {
if ( if (
resultsContainerRef.current && resultsContainerRef.current &&
(resultsContainerRef.current.contains(event.relatedTarget) || inputRef.current?.contains(event.relatedTarget)) (resultsContainerRef.current.contains(event.relatedTarget) ||
inputRef.current?.contains(event.relatedTarget))
) { ) {
event.stopPropagation(); // Prevent the blur event from triggering if focus is inside results container or input event.stopPropagation(); // Prevent the blur event from triggering if focus is inside results container or input
} else { } else {
...@@ -198,9 +200,7 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -198,9 +200,7 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
const inputTypeArray = Array.isArray(inputType) ? inputType : [inputType]; const inputTypeArray = Array.isArray(inputType) ? inputType : [inputType];
const resultType = getResultTypeHeader(data); const resultType = getResultTypeHeader(data);
if (data.input_type === "invalid_input") { if (data.input_type === "invalid_input") {
return ( return <div className="px-4 py-2">Account not found: {searchTerm}</div>;
<div className="px-4 py-2">Account not found: {searchTerm}</div>
);
} else if ( } else if (
inputTypeArray.includes(data.input_type) || inputTypeArray.includes(data.input_type) ||
(inputTypeArray.includes("account_name") && (inputTypeArray.includes("account_name") &&
...@@ -219,13 +219,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -219,13 +219,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
<div <div
key={index} key={index}
ref={selectedResult === index ? selectedResultRef : null} ref={selectedResult === index ? selectedResultRef : null}
className={cn( className={cn("autocomplete-result-item", {
"autocomplete-result-item", "bg-navbar-listHover": selectedResult === index,
{ "autocomplete-result-item": index > 0,
"bg-navbar-listHover": selectedResult === index, })}
"autocomplete-result-item": index > 0,
}
)}
onClick={() => { onClick={() => {
onChange(account); onChange(account);
inputRef.current?.focus(); inputRef.current?.focus();
...@@ -282,9 +279,14 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -282,9 +279,14 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
{capitalizeFirst(resultType)}&nbsp; {capitalizeFirst(resultType)}&nbsp;
</span> </span>
)} )}
<Link href={href} data-testid=""> <Link
href={href}
data-testid=""
>
<span className="autocomplete-result-link"> <span className="autocomplete-result-link">
{data.input_type === "transaction_hash" ? data.input_value.slice(0, 10) : data.input_value} {data.input_type === "transaction_hash"
? data.input_value.slice(0, 10)
: data.input_value}
</span> </span>
</Link> </Link>
</> </>
...@@ -310,7 +312,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -310,7 +312,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
}; };
return ( return (
<div ref={searchContainerRef} className={cn("relative", className)}> <div
ref={searchContainerRef}
className={cn("relative", className)}
>
<div className="flex items-center pr-2 z-50"> <div className="flex items-center pr-2 z-50">
<Input <Input
ref={inputRef} ref={inputRef}
...@@ -325,7 +330,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({ ...@@ -325,7 +330,10 @@ const AutocompleteInput: React.FC<AutocompleteInputProps> = ({
onKeyDown={handleKeyDown} onKeyDown={handleKeyDown}
/> />
{value && !!value.length ? ( {value && !!value.length ? (
<X className="cursor-pointer" onClick={() => resetSearchBar()} /> <X
className="cursor-pointer"
onClick={() => resetSearchBar()}
/>
) : linkResult ? ( ) : linkResult ? (
<Search /> <Search />
) : null} ) : null}
......
...@@ -42,6 +42,7 @@ const Comments: React.FC = () => { ...@@ -42,6 +42,7 @@ const Comments: React.FC = () => {
if (!commentSearchProps) { if (!commentSearchProps) {
setCommentSearchPropsFromUrl(router.query); setCommentSearchPropsFromUrl(router.query);
} }
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [commentSearchProps, router.query]); }, [commentSearchProps, router.query]);
return ( return (
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment