jaibhedia / ChainzRegistry

NFT Based Land Records Management
https://noplaybutton-41656.web.app/
MIT License
8 stars 16 forks source link

Removed the UI overflow on main page and Wallet Connect page #41

Closed a-m-a-nkumar closed 4 months ago

a-m-a-nkumar commented 4 months ago

Removed the UI overflow to make the ui more responsive.

Screenshot (218)

Screenshot (220)

Here is the screenshot of page after the edit.

image

Screenshot (223)

a-m-a-nkumar commented 4 months ago

@jaibhedia I have solved the issue #35 . Please review.

jaibhedia commented 4 months ago

@a-m-a-nkumar Thank you for the PR, I have seen you have made some other changes as well, can you just document it.

a-m-a-nkumar commented 4 months ago

@jaibhedia should i share a document listing all the changes or just mention all my work here in the comment section??

jaibhedia commented 4 months ago

Sure.

On Sat, Jun 1, 2024, 12:34 PM a-m-a-nkumar @.***> wrote:

@jaibhedia https://github.com/jaibhedia should i share a document listing all the changes or just mention all my work here in the comment section??

— Reply to this email directly, view it on GitHub https://github.com/jaibhedia/ChainzRegistry/pull/41#issuecomment-2143333491, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANZY55NCZD3ZALTYAABCXE3ZFFXBFAVCNFSM6AAAAABIIBAMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGMZTGNBZGE . You are receiving this because you were mentioned.Message ID: @.***>

jaibhedia commented 4 months ago

You can mention it here in the comments as well, no issues.

On Sat, Jun 1, 2024, 12:35 PM SS @.***> wrote:

Sure.

On Sat, Jun 1, 2024, 12:34 PM a-m-a-nkumar @.***> wrote:

@jaibhedia https://github.com/jaibhedia should i share a document listing all the changes or just mention all my work here in the comment section??

— Reply to this email directly, view it on GitHub https://github.com/jaibhedia/ChainzRegistry/pull/41#issuecomment-2143333491, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANZY55NCZD3ZALTYAABCXE3ZFFXBFAVCNFSM6AAAAABIIBAMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGMZTGNBZGE . You are receiving this because you were mentioned.Message ID: @.***>

a-m-a-nkumar commented 4 months ago

Home Page UI Overflow Fix:

Problem: The Home page had a UI overflow issue on the right side, causing a poor user experience. Solution: To address this, I used a SingleChildScrollView widget with horizontal scrolling enabled. This allows the content to be properly contained and scrollable horizontally, eliminating the overflow issue. I also made changes to the header section of the home page.

Login Section UI Improvement:

Problem: The Login section had an issue where the UI overflow caused the Metamask login button to be hidden behind an error message. This made it difficult for users to access the login functionality. Solution: To resolve this, I removed the error message that was obstructing the Metamask login button. This adjustment makes the UI more responsive and ensures that the login button is easily accessible.

Files Modified:

lib/screens/home_page.dart
lib/screens/wallet_connect.dart
lib/widget/header.dart
lib/widget/homePageDecription.dart

Other modifications include updating and including some packages.