Skip to content

Commit

Permalink
Micro changes
Browse files Browse the repository at this point in the history
  • Loading branch information
petrvecera committed Sep 27, 2024
1 parent 0a2496c commit a1c172d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion screens/leaderboards/leaderboards.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ const Leaderboards = ({
/>
<Select
withCheckIcon={false}
label="Race"
label="Faction"
style={{ width: 190 }}
defaultValue={raceToFetch}
value={raceToFetch}
Expand Down
5 changes: 3 additions & 2 deletions screens/players/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ const PlayerCard = ({
playerStatsData: ProcessedCOHPlayerStats | undefined;
replaysData: ProcessedReplayData;
}) => {
const { push, query, asPath } = useRouter();
const { push, query, asPath, replace } = useRouter();
const { view } = query;

const playerData = playerDataAPI;
Expand All @@ -101,7 +101,8 @@ const PlayerCard = ({
const originalQuery = originalUrl.searchParams;
const newURL = new URL(`/players/${playerID}/${cleanName}`, window.location.origin);
newURL.search = originalQuery.toString();
push(newURL.toString(), undefined, { shallow: true });
// Replace works the same way as push but doesn't add it into the history
replace(newURL.toString(), undefined, { shallow: true });
}
}, [playerID]);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ const ActivityLastMonthsWidget = ({
const { colorScheme } = useMantineColorScheme();
const theme = useMantineTheme();

console.log(theme.colors);

// This is most likely a bug in Nivo, the labels on Calendar Canvas our outside / other charts have it inside
const chartColorTheme = {
...getNivoTooltipTheme(colorScheme),
Expand Down

0 comments on commit a1c172d

Please sign in to comment.