-
Notifications
You must be signed in to change notification settings - Fork 4
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
Dapp update #42
Dapp update #42
Conversation
0xExp-po
commented
Sep 10, 2024
- update UI
Co-authored-by: Pamphile Roy <[email protected]>
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.
Good now it sticks to the bottom better, but there is now always a scroll bar. Can you make it fit the whole screen without that?
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 don't have such error
could you share screenshot of it please?
dapp/src/constants/demoConfigData.js
Outdated
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.
Please remove that part for this PR. The redesign of the home page is for just after. You can start another branch and PR already if you want.
For the mock data I will provide you a small list with this format. Use this for now but I don't want to have random things on the live page
I add dropdown, search box and register button
|
Thanks for the updates. Let's get this in and I will follow up for the UI 😃! |