Skip to content

Commit 6f227d5

Browse files
committed
Fix
1 parent dd52aeb commit 6f227d5

File tree

1 file changed

+0
-38
lines changed

1 file changed

+0
-38
lines changed

src/components/Header.tsx

Lines changed: 0 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,9 @@ import useStyles from "@/styles/jss/components/header";
1818

1919
type ColorExpanded = PropTypes.Color | "transparent";
2020

21-
interface ChangeColorOnScroll {
22-
color: ColorExpanded;
23-
height: string | number;
24-
}
25-
2621
interface HeaderProps {
2722
absolute?: string;
2823
brand?: string;
29-
changeColorOnScroll?: ChangeColorOnScroll;
3024
color?: ColorExpanded;
3125
fixed?: boolean;
3226
rightLinks?: ReactElement;
@@ -37,49 +31,17 @@ function Header(props: HeaderProps): ReactElement {
3731
const [mobileOpen, setMobileOpen] = useState(false);
3832
const [repositoryData] = useRepository();
3933

40-
useEffect(() => {
41-
if (props.changeColorOnScroll) {
42-
window.addEventListener("scroll", headerColorChange);
43-
}
44-
return function cleanup() {
45-
if (props.changeColorOnScroll) {
46-
window.removeEventListener("scroll", headerColorChange);
47-
}
48-
};
49-
}, []);
50-
5134
function handleDrawerToggle(): void {
5235
setMobileOpen(!mobileOpen);
5336
}
5437

55-
function headerColorChange(): void {
56-
const { color, changeColorOnScroll } = props;
57-
const windowsScrollTop = window.pageYOffset;
58-
if (windowsScrollTop > changeColorOnScroll.height) {
59-
document.body
60-
.getElementsByTagName("header")[0]
61-
.classList.remove(classes[color]);
62-
document.body
63-
.getElementsByTagName("header")[0]
64-
.classList.add(classes[changeColorOnScroll.color]);
65-
} else {
66-
document.body
67-
.getElementsByTagName("header")[0]
68-
.classList.add(classes[color]);
69-
document.body
70-
.getElementsByTagName("header")[0]
71-
.classList.remove(classes[changeColorOnScroll.color]);
72-
}
73-
}
74-
7538
const { color, rightLinks, brand, fixed, absolute } = props;
7639

7740
const theme = useTheme();
7841
return (
7942
<AppBar
8043
className={clsx({
8144
[classes.appBar]: true,
82-
[classes[color]]: color,
8345
[classes.absolute]: absolute,
8446
[classes.fixed]: fixed,
8547
})}

0 commit comments

Comments
 (0)