Settings page boiler plate #18
Conversation
|
Someone is attempting to deploy a commit to the Tapped Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| import Component from "../../components/ui/toggle"; | ||
| const Settings = () => { | ||
| return ( | ||
| <div className="min-h-screen p-8"> |
There was a problem hiding this comment.
we use react-form-hooks and zod lib for making forms like. there isn't any of that code in here rn
| <h1 className="text-3xl mb-8 text-center">Settings</h1> | ||
| <div className="flex flex-col items-center mb-8"> | ||
| <div className="relative"> | ||
| <img style={{ marginLeft: "27%" }} src="https://via.placeholder.com/150" alt="Profile" className="w-32 h-32 rounded-full object-cover" /> |
There was a problem hiding this comment.
nextjs has an component that shuld be used instead of raw
| </div> | ||
| </div> | ||
| <h2 className="text-xl mb-4">Preferences</h2> | ||
| <button style={{ marginLeft: "85%", width: "5rem" }} className="p-2 bg-blue-500 rounded-md text-white cursor-pointer hover:bg-blue-600 focus:outline-none focus:ring-2 focus:ring-blue-500"> |
There was a problem hiding this comment.
these raw styles mixed with tailwindcss gives me nightmares.
also, it doesn't use the shadcn component we already have
| <input type="text" id="username" name="username" defaultValue="mdotr" className="w-full p-3 bg-gray-700 rounded-md text-white focus:outline-none focus:ring-2 focus:ring-blue-500" required /> | ||
| </div> | ||
| <div className="mb-6"> | ||
| <label htmlFor="bio" className="block text-gray-400 mb-2">Bio</label> |
There was a problem hiding this comment.
these tags and whatnot are handled by the shadcn
and components we already have
| <label htmlFor="bio" className="block text-gray-400 mb-2">Bio</label> | ||
| <input type="text" id="bio" name="bio" placeholder="Enter your bio" className="w-full p-3 bg-gray-700 rounded-md text-white focus:outline-none focus:ring-2 focus:ring-blue-500" /> | ||
| </div> | ||
| <div className="mb-6"> |
There was a problem hiding this comment.
links should be instead of in nextjs
also, we already have a prebuilt city search field
| @@ -0,0 +1,66 @@ | |||
| export default function Component() { | |||
There was a problem hiding this comment.
we already have a prebuilt theme toggle
No description provided.