Rename next() and previous() to descriptive names

stepandra · Feb 20, 2023 · 54697b8 · 54697b8
1 parent a639760
commit 54697b8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/orbitez/components/ServerSelector/ServerSelector.jsx b/orbitez/components/ServerSelector/ServerSelector.jsx
@@ -14,8 +14,8 @@ const ServerSelector = () => {
         isLoading,
         contractServers,
         selectedServerIndex,
-        next,
-        previous
+        selectNextServer,
+        selectPreviousServer
     } = useContractServersContext();
 
     const loadingStateContent = useMemo(
@@ -43,7 +43,7 @@ const ServerSelector = () => {
         return (
             <>
                 <button
-                    onClick={next}
+                    onClick={selectNextServer}
                     className="server-selector__button"
                     disabled={areButtonsDisabled}
                 >
@@ -72,7 +72,7 @@ const ServerSelector = () => {
                     </ReactTooltip>
                 )}
                 <button
-                    onClick={previous}
+                    onClick={selectPreviousServer}
                     className="server-selector__button"
                     disabled={areButtonsDisabled}
                 >

diff --git a/orbitez/hooks/useContractServers.js b/orbitez/hooks/useContractServers.js
@@ -127,7 +127,7 @@ const useContractServers = () => {
         setStatsUrl(selectedServerStatsUrl);
     }, [contractServers, selectedServerIndex]);
 
-    const next = useCallback(() => {
+    const selectNextServer = useCallback(() => {
         if (contractServers.length === selectedServerIndex + 1) {
             setSelectedServerIndex(0);
             return;
@@ -136,7 +136,7 @@ const useContractServers = () => {
         setSelectedServerIndex(selectedServerIndex + 1);
     }, [contractServers, selectedServerIndex]);
 
-    const previous = useCallback(() => {
+    const selectPreviousServer = useCallback(() => {
         if (selectedServerIndex === 0) {
             setSelectedServerIndex(contractServers.length - 1);
             return;
@@ -154,8 +154,8 @@ const useContractServers = () => {
         isLoading,
         contractServers,
         selectedServerIndex,
-        next,
-        previous
+        selectNextServer,
+        selectPreviousServer
     }
 };