Conversation
src/@types/custom.d.ts
Outdated
| @@ -0,0 +1,5 @@ | |||
| /* eslint-disable @typescript-eslint/no-explicit-any */ | |||
| declare module '*.svg' { | |||
| const content: any; | |||
There was a problem hiding this comment.
i don't like any types...
src/components/VslSection.tsx
Outdated
| .big-fragments-group { | ||
| left: 75%; | ||
| top: -30%; | ||
| /* top: -30%; */ |
src/pages/index.tsx
Outdated
| function IndexPage() { | ||
| return ( | ||
| <Box width="100%"> | ||
| <Box w="100vw" h="100vh" bgColor="#0B132B"> |
There was a problem hiding this comment.
please, don't use vw or vh here
leticiabsduarte
left a comment
There was a problem hiding this comment.
There is a mobile version being made, but good work
|
❌ Deploy Preview for lucid-shirley-299a0a failed. 🔨 Explore the source changes: 909e10f 🔍 Inspect the deploy log: https://app.netlify.com/sites/lucid-shirley-299a0a/deploys/620561644a74bb000813edac |
Manuel-Antunes
left a comment
There was a problem hiding this comment.
Remove all SCSS files, use styled components
…eated the navigation pages
| } | ||
| `; | ||
|
|
||
| const ContainerStyled = styled(Container)` |
There was a problem hiding this comment.
Why not change container to fluid, add flex using bootstrap class instead of use styled components?
| box-shadow: 0px 0px 12px #cf721c; | ||
| `; | ||
|
|
||
| export default Navbar; |
There was a problem hiding this comment.
All classes can be overwriten using some bootstrap classes
| export const Main: React.FC = ({ children }) => { | ||
| const href = useMemo(() => (window ? window.location.href : ''), [window]); | ||
|
|
||
| const path = href.slice(href.lastIndexOf('/') + 1); |
There was a problem hiding this comment.
add all this logics in memo, we uses memo to optimize updatable variables, so, if we need to change things use the memo hook, click here for more information
Related Issue
Proposed Changes
Additional Info
Checklist
Screenshots