Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor CSS Fixes (sticky nav and layout) #299

Merged
merged 8 commits into from
Apr 9, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 5 additions & 8 deletions src/components/layout/Layout.module.css
Original file line number Diff line number Diff line change
@@ -7,11 +7,11 @@
.main {
flex: 10;
position: relative;
background-color: var(--background);
color: var(--on-background);
height: 100%;
width: 100%;
padding: 5em 3em 3em 3em;
margin-left: 0;
overflow: auto;
}

.mainPurple {
@@ -30,12 +30,6 @@
background-color: var(--grey-80);
}

/* @media only screen and (max-width: 480px) {
.main {
padding: 5em 3em 3em 3em;
}
} */

@media only screen and (min-width: 480px) {
.main {
padding: 5em 3em 3em 3em;
@@ -51,6 +45,9 @@

/* Medium Devices, Desktops */
@media only screen and (min-width: 992px) {
.main {
margin-left: 200px;
}
}

/* Large Devices, Wide Screens */
3 changes: 3 additions & 0 deletions src/components/navbar/Navbar.module.css
Original file line number Diff line number Diff line change
@@ -18,6 +18,9 @@
background-color: var(--primary-40);
display: block;
width: 200px;
position: fixed;
z-index: 1;
height: 100vh;
}

.link {
2 changes: 1 addition & 1 deletion src/styles/StudentPage.module.css
Original file line number Diff line number Diff line change
@@ -58,7 +58,7 @@
border-radius: 10px;
margin-top: 2rem;
min-height: 50vh;
max-height: 70vh;
max-height: 100vh;
}

.noIepBox {

Unchanged files with check annotations Beta

import { trpc } from "@/client/lib/trpc";
import Goals from "@/components/goal/Goal";
import $button from "@/components/design_system/button/Button.module.css";
import $input from "@/styles/Input.module.css";

Check warning on line 4 in src/components/iep/Iep.tsx

GitHub Actions / lint

'$input' is defined but never used
import Box from "@mui/material/Box";
import Container from "@mui/material/Container";
import List from "@mui/material/List";
import { trpc } from "@/client/lib/trpc";

Check warning on line 1 in src/components/subgoal/Subgoal.tsx

GitHub Actions / lint

'trpc' is defined but never used
import { Subgoal } from "@/types/global";
import Box from "@mui/material/Box";
import Button from "@mui/material/Button";
{currentStep === UploadStep.UploadPicture && (
<div className={styles.uploadContainer}>
{selectedFile && (
<img

Check warning on line 120 in src/components/uploadPicture/uploadImage.tsx

GitHub Actions / lint

Using `<img>` could result in slower LCP and higher bandwidth. Use `<Image />` from `next/image` instead to utilize Image Optimization. See: https://nextjs.org/docs/messages/no-img-element
src={URL.createObjectURL(selectedFile)}
className={styles.imagePreview}
alt="preview of image"
{data ? (
<>
{data.content_type.startsWith("image/") ? (
<img

Check warning on line 26 in src/components/uploadedFile/uploadedFile.tsx

GitHub Actions / lint

Using `<img>` could result in slower LCP and higher bandwidth. Use `<Image />` from `next/image` instead to utilize Image Optimization. See: https://nextjs.org/docs/messages/no-img-element
src={data.url}
className={styles.image}
alt={`user uploaded image: ${data.name}`}
import { useState } from "react";
import { trpc } from "@/client/lib/trpc";
import { useRouter } from "next/router";
import $home from "@/styles/Home.module.css";

Check warning on line 4 in src/pages/staff/[user_id].tsx

GitHub Actions / lint

'$home' is defined but never used
import $button from "@/components/design_system/button/Button.module.css";
import $StaffPage from "../../styles/StaffPage.module.css";
import $Modal from "../../styles/Modal.module.css";
"0px 4px 4px 0px rgba(0, 0, 0, 0.30), 0px 8px 12px 6px rgba(0, 0, 0, 0.15)",
...new Array(20).map(() => "none"),
// type requires 20+ elements
] as any,

Check warning on line 17 in src/theme.ts

GitHub Actions / lint

Unexpected any. Specify a different type
typography: {
allVariants: {
letterSpacing: "normal",