Skip to content

Comments

feat: polish ui#971

Merged
43081j merged 2 commits intonpmx-dev:mainfrom
alexdln:feat/polish-ui
Feb 4, 2026
Merged

feat: polish ui#971
43081j merged 2 commits intonpmx-dev:mainfrom
alexdln:feat/polish-ui

Conversation

@alexdln
Copy link
Member

@alexdln alexdln commented Feb 4, 2026

Improvements:

  • The logo position on the main page (was shifted slightly to the right of the center)
  • Micro-animation for a logo
  • Extended colors for dot in logo "./"
  • Fixed the rounding in the menu for connected options
  • Micro-animations in search boxes (incl border color on hover)
  • Fixed border for btns
  • Improved indents in home page (equal and big indents didn't create content groups)
  • Fixed the sizes and paddings of the search field and button
  • Hidden the "search" text from the search button on mobile, as it took up third the field
Some screenshots Home Page: image image

Mobile:
image

image

Hover:
image

Closes #969

@vercel
Copy link

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 4, 2026 10:56pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 4, 2026 10:56pm
npmx-lunaria Ignored Ignored Feb 4, 2026 10:56pm

Request Review

@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
app/pages/index.vue 66.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

This pull request makes visual and accessibility-focused adjustments across multiple components: AppHeader (spacing and hover colour), AppLogo (SVG fill to currentColor), AccountMenu (added rounded-md to menu buttons), SearchBox (icon and input hover/focus/outline and transition refinements), Tag Button (added border-solid), and the index page (header padding, margins, logo positioning, search input sizing and accessibility label changes). No logic, event handling, or exported/public API signatures were changed.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly outlines specific UI improvements matching the changeset: logo positioning, animations, color extensions, menu rounding fixes, search box animations, border fixes, indent improvements, and responsive sizing adjustments.
Linked Issues check ✅ Passed All code changes align with the linked issue #969 objectives: the PR implements UI improvements across sizes, margins, transitions, consistency, colours, and responsiveness as specified.
Out of Scope Changes check ✅ Passed All modifications are scoped to UI improvements and refinements as defined in issue #969. No unrelated or extraneous code changes detected.

✏️ 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

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

Copy link
Contributor

@43081j 43081j left a comment

Choose a reason for hiding this comment

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

checked each one and all looks better to me!

very nice work. if you're interested in more UX too, there's #776

@43081j 43081j added this pull request to the merge queue Feb 4, 2026
Merged via the queue into npmx-dev:main with commit 8483ee4 Feb 4, 2026
16 checks passed
@alexdln alexdln deleted the feat/polish-ui branch February 4, 2026 23:23
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.

feat: polish ui

2 participants