Skip to content

Update PillarX App Download Link#485

Merged
ashutoshamazix merged 2 commits intostagingfrom
chore/update-app
Dec 22, 2025
Merged

Update PillarX App Download Link#485
ashutoshamazix merged 2 commits intostagingfrom
chore/update-app

Conversation

@ashutoshamazix
Copy link
Collaborator

@ashutoshamazix ashutoshamazix commented Dec 22, 2025

Description

  • Added PillarX App Download Link on the Landing/Home page

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Summary by CodeRabbit

  • Bug Fixes

    • Enabled the download section to be visible and functional
    • Corrected App Store and Google Play download links to point to actual app listings
  • Design

    • Improved vertical spacing on mobile layouts for better visual hierarchy

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 22, 2025

Walkthrough

Landing page styling and app store links are updated. CSS padding values increase from 5x to 8x multiplier on small screens, the download section visibility is enabled by removing display: none rules, and Apple App Store and Google Play Store links are changed to direct app listing pages.

Changes

Cohort / File(s) Change Summary
Landing page styling
src/components/LandingPage/GlobalStyles/index.jsx
Increased vertical padding in wrappers from calc(var(--padding) * 5) to calc(var(--padding) * 8) on small screens; removed display: none rules from home_app__download and preceding selector, enabling download section visibility; added extra bottom padding to hero wrappers
App store links
src/pages/Landing.jsx
Updated Apple App Store link from generic store URL to specific app listing (id1346582238); updated Google Play Store link from generic store URL to specific wallet app listing (com.pillarproject.wallet)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Verify app store URLs point to correct app listings and are not subject to change
  • Confirm CSS spacing changes render correctly on small screens through visual testing

Possibly related PRs

Suggested reviewers

  • IAmKio
  • RanaBug

Poem

🐰 Hop, skip, and a bound—the landing page leaps!
With padding so tall and links running deep,
The download section wakes from its sleep,
To Apple and Google's app stores we creep! 📱✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete; testing section is empty, no screenshots provided, and no change type is selected despite clear modifications to code. Add testing details (manual testing, environment), select appropriate change type (likely 'New feature'), and include screenshots if UI changes are visible.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: updating the PillarX app download links in the Landing page.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-app

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 16fc28c and 0332b8a.

⛔ Files ignored due to path filters (1)
  • src/containers/__snapshots__/Main.test.tsx.snap is excluded by !**/*.snap
📒 Files selected for processing (2)
  • src/components/LandingPage/GlobalStyles/index.jsx
  • src/pages/Landing.jsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: unit-tests
  • GitHub Check: lint
  • GitHub Check: Cloudflare Pages: pillarx-debug
🔇 Additional comments (2)
src/components/LandingPage/GlobalStyles/index.jsx (1)

793-793: Verify the visual impact of increased mobile padding.

The vertical padding for hero sections on small screens has increased by 60% (from 5 to 8 multiplier). Ensure this adjustment doesn't push critical content below the fold or create excessive whitespace on mobile devices.

Also applies to: 2707-2707

src/pages/Landing.jsx (1)

318-329: Both app store links correctly resolve to the official PillarX wallet application. Apple App Store (id1346582238) points to PillarX: Smart DeFi Wallet by Pillar Project, and Google Play Store (com.pillarproject.wallet) hosts the Pillar Multichain DeFi Wallet with migration support to PillarX. The app IDs and URLs are correct.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ashutoshamazix ashutoshamazix merged commit c625413 into staging Dec 22, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants