-
Notifications
You must be signed in to change notification settings - Fork 15
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
create layout #456
create layout #456
Conversation
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.
A few comments:
- please setup your linter as there are some formatting issues. You can see them when you run
npm run lint
. You can install Prettier and ESLint to automate this - there are some responsivness issues, please ensure the design work for all viewports (the smallest one 320px width mobile screen).
To address the responsiveness, I suggest to have a centered container of 800px that contains 2 400px cols. But when screenwidth is below 600px switches to having the content stacked in a single column again.
Other than that, great work!
fixed all error |
This is because an error occurs during build.
… Message ID: ***@***.***
com>
|
***@***.***/tailwindcss-scrollbar
…On Tue, Jun 18, 2024 at 5:44 PM dcts ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In package.json
<#456 (comment)>
:
> @@ -40,6 +40,7 @@
"react-redux": "^8.1.2",
"resize-observer-polyfill": "^1.5.1",
"tailwindcss": "3.3.2",
+ "tailwindcss-scrollbar": "^0.1.0",
Can you send a link to the NPM package used here?
—
Reply to this email directly, view it on GitHub
<#456 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A77GYERRWMKOZ6JO6DFJJ7TZICZ5BAVCNFSM6AAAAABJL2DV6WVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMRWGY4DOOJVGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
i am waiting for your new feedback |
|
Great work, thank you for fixing all issues! |
No description provided.