#6262 sidebar consistency

[prachi00]

aThank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

👇 __ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Needs QA check

• @kodadot/qa-guild please review

Context

• Closes Sidebar consistency  #6262
• Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

• My DOT address: Payout

Screenshot 📸

• My fix has changed UI

Copilot Summary

[kodabot]

WARNING @prachi00 PR for issue #6262 which isn't assigned to you. Please be warned that this PR may get rejected if there's another assignee for issue #6262

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	1781770
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/6492301a8a68d00008dca8b0
😎 Deploy Preview	https://deploy-preview-6287--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[reviewpad]

AI-Generated Summary: This pull request addresses issue #6262 by improving sidebar consistency across multiple components. It updates the ConnectWalletModal and NotificationBoxModal titles to use the same CSS classes, and moves the .modal-card-title styling from the _connect-wallet.scss file to apply globally. The changes made involve 3 files with 8 insertions and 8 deletions.

[prury]

They are all about the same height now, good.

@prachi00, when you changed the Connect Wallet sidebar to Connect Wallet, you also changed the profile to Connect wallet:

[exezbcz]

yup, works for me once the connect wallet profile is changed

thank you!

[prachi00]

They are all about the same height now, good.

@prachi00, when you changed the Connect Wallet sidebar to Connect Wallet, you also changed the profile to Connect wallet:

arent they the same sidebar modal?

[exezbcz]

@prachi00 yes, but connect wallet is not what the profile sidebar is about

[prury]

They are all about the same height now, good.
@prachi00, when you changed the Connect Wallet sidebar to Connect Wallet, you also changed the profile to Connect wallet:

arent they the same sidebar modal?

they are, but their states are different depending if the user has his/her wallet connected or not, for example:

• the title here should be "connect wallet":
  

and once the user connects its wallet, it should be renamed to Profile:

[prachi00]

@prury got it, fixed

[codeclimate]

Code Climate has analyzed commit 1781770 and detected 0 issues on this pull request.

View more on Code Climate.