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
Restyle account page #858
Restyle account page #858
Conversation
const backgroundColor = useColorModeValue("white", "gray.700"); | ||
|
||
const BoxClasses: BoxProps = { | ||
gap: "2", | ||
borderRadius: "xl", | ||
shadow: "base", | ||
className: "p-4 sm:p-6", | ||
className: "p-4 sm:p-6 " + (props.className ?? ""), |
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.
You can use clsx
here to simplify the code
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.
Of cousre!
import Head from "next/head"; | ||
import Link from "next/link"; | ||
import { useSession } from "next-auth/react"; | ||
import React from "react"; | ||
export { getDefaultStaticProps as getStaticProps } from "src/lib/default_static_props"; | ||
import { MdOutlineEdit } from "react-icons/md"; |
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.
We are trying to use one type of icon. Can you switch to Feather icon here?
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.
I am not sure we talked about that.
is Feather icon installed ?
I will leave it to a different PR anyway
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.
I've been using Font Awesome (react-icons/fa
) but they don't don't have an outline edit icon. I think this is okay for now.
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.
I say icons can be fixed later.
Related #747