fix: menu buttons resseting scroll position on mobile

[stephenjason89]

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Menu buttons resseting scroll position on mobile #6667 Notification sidebar resets scroll position on mobile #6669 Cart sidebar resets scroll position on mobile #6668
• Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

• My contribution builds clean without any errors or warnings
• I've merged recent default branch -- main and I've no conflicts
• I've tried to respect high code quality standards
• I've didn't break any original functionality

Optional

• I've tested it at </ksm/collection>
• I've tested PR on mobile
• I've written unit tests 🧪
• I've found edge cases

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

• Fill up your DOT address: Payout

Community participation

• Are you at KodaDot Discord?

Screenshot 📸

• My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at 769abca

Refactored the code for opening the shopping cart modal from the navbar. Moved the logic and configuration from ShoppingCartModalConfig.ts and ShoppingCartButton.vue to Navbar.vue to improve code organization and performance.

🤖 Generated by Copilot at 769abca

ShoppingCartButton
Simpler, no more emit -
Autumn leaves falling

[kodabot]

SUCCESS @stephenjason89 PR for issue #6667 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime!

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	704b766
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/64d605c757541c00086f5e1d
😎 Deploy Preview	https://deploy-preview-6673--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 configuration.

[reviewpad]

AI-Generated Summary: This pull request contains changes to the Navbar.vue component to fix an issue where on mobile, the menu buttons were resetting the scroll position. The fix involved removing two lines of code. The first line removed a Vue directive v-if="account" from the NotificationBoxButton component. The second line removed was the command document.body.classList.toggle('is-clipped') which was toggling a CSS class on the body. Removing this line prevents the toggling of this class which in turn prevents the scroll position from being reset.

[roiLeo]

https://github.com/kodadot/nft-gallery/pull/6673/files#r1291063287

[codeclimate]

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

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[roiLeo]

code lgtm

[roiLeo]

waiting for @prury review

[prury]

Notifications ✔️
Shopping Cart ✔️
Profile ✔️

For chain switching lets wait for #6613

[yangwao]

pay 60 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $60 @ 5.03 USD/DOT ~ 11.928 $DOT
🧗 16UcV9V6nVvPYdHz98ymUKmNLkzjCEU5sbKJMi7hxYyTHjzR
🔗 0x7a70f99824f21f6d60bdb3c7a4e993ed935c0bd6b41f9b7c844f556e69fabc25

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues