Skip to content

task: improve marketplace button design#801

Merged
coodos merged 2 commits intomainfrom
task/improve-marketplace-btn
Feb 13, 2026
Merged

task: improve marketplace button design#801
coodos merged 2 commits intomainfrom
task/improve-marketplace-btn

Conversation

@Bekiboo
Copy link
Copy Markdown
Collaborator

@Bekiboo Bekiboo commented Feb 11, 2026

Description of change

add marketplace image and W3DS logo SVG
improve marketplace button design

Issue Number

Type of change

  • Update (a change which updates existing functionality)

How the change has been tested

Change checklist

  • I have ensured that the CI Checks pass locally
  • I have removed any unnecessary logic
  • My code is well documented
  • I have signed my commits
  • My code follows the pattern of the application
  • I have self reviewed my code

Summary by CodeRabbit

  • UI/UX Improvements
    • Replaced the simple marketplace button with a rich, image-backed hero-style banner featuring a background image, gradient overlay, layered text/icons, and drop shadows.
    • Updated button content to a two-line layout: prominent title "Discover Post Platforms" and secondary line "Explore Marketplace" with embedded logos/icons, aligned toward the bottom for a card-like appearance.

@Bekiboo Bekiboo self-assigned this Feb 11, 2026
@Bekiboo Bekiboo requested a review from coodos as a code owner February 11, 2026 13:38
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

Replaces the previous simple marketplace link button in the main page with a layered, image-backed hero-style card: absolute background image, gradient overlay, stacked text/icons (two-line title and subtitle with W3DS logo), and updated layout/visual structure while keeping the same navigation target and attributes.

Changes

Cohort / File(s) Summary
Main page UI
infrastructure/eid-wallet/src/routes/(app)/main/+page.svelte
Replaced a plain rounded marketplace Button.Nav with a rich hero-style card: added absolute background image, gradient overlay, z-indexed text, two-line title ("Discover Post Platforms") and subtitle ("Explore Marketplace" with W3DS logo), removed previous inline span/icon and adjusted layout/styling. (+31/-8)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • coodos

Poem

🐰 I found a card with image bright,
Layers stacked in soft twilight,
"Discover" calls in fonts that sing,
A tiny logo takes to wing,
Hooray — the marketplace takes flight! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change—improving the marketplace button design—which directly aligns with the primary objective of replacing a simple button with a rich, image-backed hero-style card.
Description check ✅ Passed The description covers the required template sections with most information provided, but the 'How the change has been tested' section is empty, which is a notable gap in documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch task/improve-marketplace-btn

No actionable comments were generated in the recent review. 🎉


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.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@infrastructure/eid-wallet/src/routes/`(app)/main/+page.svelte:
- Line 75: Remove the leftover debug console.log by deleting the statement
console.log("status current", profileCreationStatus) in the +page.svelte
component so no debug output remains; if keeping visibility is required, replace
it with a proper logger or reactive UI binding that uses the
profileCreationStatus variable instead of a console.log.
- Around line 62-70: The onMount async IIFE that awaits
globalState.userController.user, globalState.userController.isFake and
globalState.vaultController.vault has no error handling, so rejections are
swallowed and userData/ename remain undefined; wrap the async work in a
try/catch (or attach .catch) around the awaits inside the onMount handler, log
or surface the error via a visible UI state or processLogger, and set a safe
fallback for userData and ename to avoid undefined values (reference onMount,
userController.user, userController.isFake, vaultController.vault, userData,
ename).
🧹 Nitpick comments (1)
infrastructure/eid-wallet/src/routes/(app)/main/+page.svelte (1)

13-14: Consolidate svelte imports into a single statement.

onMount and onDestroy are imported from "svelte" in two separate lines.

Proposed fix
-    import { type Snippet, getContext, onMount } from "svelte";
-    import { onDestroy } from "svelte";
+    import { type Snippet, getContext, onDestroy, onMount } from "svelte";

Comment thread infrastructure/eid-wallet/src/routes/(app)/main/+page.svelte Outdated
Comment thread infrastructure/eid-wallet/src/routes/(app)/main/+page.svelte Outdated
@coodos coodos merged commit fde86bd into main Feb 13, 2026
4 checks passed
@coodos coodos deleted the task/improve-marketplace-btn branch February 13, 2026 16:25
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