-
Notifications
You must be signed in to change notification settings - Fork 6
Talk application page #36
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
base: master
Are you sure you want to change the base?
Changes from 1 commit
132941e
d00bf06
792c7a3
e19a4a3
22e0ba4
f611dcd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,41 @@ | ||
| import { ReactNode } from "react" | ||
| import type { Metadata } from "next" | ||
| import "../globals.css" | ||
| import { Header } from "../components/Header" | ||
| import { Footer } from "../components/Footer" | ||
| import { siteConfig } from "../siteConfig" | ||
| import StyledComponentsRegistry from "../StyledComponentsRegistry" | ||
|
|
||
| export const metadata: Metadata = { | ||
| title: siteConfig.title, | ||
| description: siteConfig.description, | ||
| openGraph: { | ||
| type: "website", | ||
| url: siteConfig.url, | ||
| title: siteConfig.title, | ||
| description: siteConfig.description, | ||
| images: siteConfig.ogImage | ||
| } | ||
| } | ||
|
|
||
| export default function RootLayout({ | ||
| children | ||
| }: Readonly<{ | ||
| children: ReactNode | ||
| }>) { | ||
| return ( | ||
| <html lang="en"> | ||
| <head> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1" /> | ||
| <link rel="icon" href="/images/favicon.png" /> | ||
| </head> | ||
| <body> | ||
| <StyledComponentsRegistry> | ||
| <Header /> | ||
| {children} | ||
| <Footer /> | ||
| </StyledComponentsRegistry> | ||
| </body> | ||
| </html> | ||
| ) | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,20 +1,22 @@ | ||
| body { | ||
| font-size: 18pt; | ||
| font-size: 18pt; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Inconsistent formatting should be solved a single commit: "Fix files with broken formatting"
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we add a universal standard to the lint rules? |
||
| } | ||
|
|
||
| form { | ||
| display: grid; | ||
| grid-template-columns: 1fr 1fr; | ||
| gap: 0.5em; | ||
| padding: 0.5em; | ||
| display: grid; | ||
| grid-template-columns: 1fr 1fr; | ||
| gap: 0.5em; | ||
| padding: 0.5em; | ||
| } | ||
|
|
||
| input, textarea, button { | ||
| font-size: inherit; | ||
| padding: 0.5em; | ||
| input, | ||
| textarea, | ||
| button { | ||
| font-size: inherit; | ||
| padding: 0.5em; | ||
| } | ||
|
|
||
| textarea { | ||
| grid-column: span 2; | ||
| resize: vertical; | ||
| } | ||
| grid-column: span 2; | ||
| resize: vertical; | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,18 +1,15 @@ | ||
| // import { getTalks } from "../api/talk-requests/route" | ||
| import "./page.css" | ||
|
|
||
| export default async function Talk() { | ||
| // const data = await getTalks() | ||
| return ( | ||
| <main> | ||
| {/* Current talks: {JSON.stringify(data)} <br /> */} | ||
| <form className="max-w-screen-md m-auto" action="/api/talk-requests" method="POST"> | ||
| <input type="text" placeholder="Speaker Name" name="speaker" /> | ||
| <input type="text" placeholder="Talk Title" name="title" /> | ||
| <textarea placeholder="Talk Description..." name="description"></textarea> | ||
| <input className="btn hover:bg-accent hover:text-base-100" type="reset" value="Reset" /> | ||
| <button className="btn hover:bg-accent hover:text-base-100">Submit</button> | ||
| </form> | ||
| </main> | ||
| <div> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| <h1>Talk</h1> | ||
| </div> | ||
|
||
| ) | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure I'm convinced we need a whole other layout file for the talk page. We should make sure our layout is flexible enough for both the main page and the other pages. Our site only has one layout design, so we should have only one layout file.