Skip to content

fix: constrain student layout height so inner scroll containers work #1463

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.

Sign up for GitHub

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

Open
Xenon010101 wants to merge 2 commits into
base: main
Choose a base branch
from
+21 −8
Open

fix: constrain student layout height so inner scroll containers work #1463

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
16937a4
fix: improve Contact Us page heading and input styling
Xenon010101 Jun 5, 2026
ec19772
fix: constrain student layout height so inner scroll containers work
Xenon010101 Jun 5, 2026
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
21 changes: 16 additions & 5 deletions client/src/module/legal/ContactPage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { useState } from "react";
import { motion } from "framer-motion";
import { Mail, Globe, Clock3, Bug, Loader2, Send, CheckCircle, AlertCircle, User, Tag, MessageSquare } from "lucide-react";
import { Navbar } from "../../components/Navbar";
import { Footer } from "../../components/Footer";
Expand Down Expand Up @@ -44,7 +45,17 @@ export default function ContactPage() {
{/* Header */}
<div className="mb-12 text-center">
<h1 className="text-4xl font-bold text-gray-900 dark:text-white mb-3">
Contact Us
Contact{" "}
<span className="relative inline-block">
<span className="relative z-10">Us</span>
<motion.span
initial={{ scaleX: 0 }}
animate={{ scaleX: 1 }}
transition={{ duration: 0.7, delay: 0.3, ease: "easeOut" }}
aria-hidden
className="absolute bottom-1 left-0 right-0 h-3 bg-lime-400 origin-left -z-0"
/>
</span>
</h1>
<p className="text-base text-gray-500 dark:text-gray-400 max-w-2xl mx-auto">
We'd love to hear from you. Reach out for support, feedback,
Expand All @@ -65,7 +76,7 @@ export default function ContactPage() {
<input
id="name" name="name" value={formData.name} onChange={handleChange} required
placeholder="Your name"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-white dark:bg-gray-800 px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-transparent px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
/>
</div>
</div>
Expand All @@ -78,7 +89,7 @@ export default function ContactPage() {
<input
id="email" name="email" type="email" value={formData.email} onChange={handleChange} required
placeholder="you@example.com"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-white dark:bg-gray-800 px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-transparent px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
/>
</div>
</div>
Expand All @@ -92,7 +103,7 @@ export default function ContactPage() {
<input
id="subject" name="subject" value={formData.subject} onChange={handleChange} required
placeholder="What's this about?"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-white dark:bg-gray-800 px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-transparent px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500"
/>
</div>
</div>
Expand All @@ -105,7 +116,7 @@ export default function ContactPage() {
<textarea
id="message" name="message" rows={5} value={formData.message} onChange={handleChange} required
placeholder="Tell us what's on your mind..."
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-white dark:bg-gray-800 px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500 resize-y"
className="w-full rounded-lg border border-gray-300 dark:border-gray-700 bg-transparent px-3 py-2 text-sm text-gray-900 dark:text-white placeholder-gray-400 focus:border-lime-500 focus:outline-none focus:ring-1 focus:ring-lime-500 resize-y"
/>
</div>
</div>
Expand Down
8 changes: 5 additions & 3 deletions client/src/module/student/StudentLayout.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export default function StudentLayout() {
}

return (
<div className="min-h-screen bg-stone-50 dark:bg-stone-950">
<div className="min-h-screen bg-stone-50 dark:bg-stone-950 flex flex-col">
<SEO title="Student Dashboard" noIndex />
{/* Navbar hidden on mobile (mobile top bar is in StudentSidebar) */}
<div className="hidden lg:block">
Expand All @@ -28,11 +28,13 @@ export default function StudentLayout() {
{sidebar}
<main
aria-label="Student main content"
className={`pt-16 lg:pt-24 px-4 pb-8 sm:px-6 lg:px-8 transition-all duration-300 overflow-auto ${
className={`flex-1 min-h-0 pt-16 lg:pt-24 px-4 pb-8 sm:px-6 lg:px-8 transition-all duration-300 overflow-hidden ${
collapsed ? "lg:ml-18" : "lg:ml-64"
}`}
>
<Outlet />
<div className="h-full overflow-y-auto">
<Outlet />
</div>
</main>
</div>
);
Expand Down
Loading