Skip to content

Conversation

@NandkishorJadoun
Copy link
Contributor

Description

  • resolved an issue in MobileTreeDrawer where icons were appearing over the sticky header. Applied a z-index to the sticky header to ensure proper stacking order.
  • removed a duplicate chevron icon from the PageSize select component to improve visual consistency.

Before & After

First Fix: Sticky Header z-Index

Before

Screencast.from.11-02-26.10.09.01.PM.IST.webm

After

Screencast.from.11-02-26.10.07.42.PM.IST.webm
Second Fix: Duplicate Chevron Removal

Before

Screenshot before fix

After

Screenshot after fix

@vercel
Copy link

vercel bot commented Feb 11, 2026

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

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Feb 11, 2026 5:02pm
npmx.dev Ready Ready Preview, Comment Feb 11, 2026 5:02pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 11, 2026 5:02pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

This pull request makes two targeted UI adjustments: the header div in MobileTreeDrawer.vue now includes an explicit z-index (z-10) to adjust stacking order relative to sibling elements, and a decorative chevron-down icon element has been removed from the page size control within PaginationControls.vue. No changes to functional behaviour, event handling, or public APIs have been introduced.

Possibly related PRs

  • npmx-dev/npmx.dev PR 1042: Makes related UI styling changes including header stacking/z-index adjustments and modifications to pagination controls components.

Suggested labels

front

Suggested reviewers

  • danielroe
  • whitep4nth3r
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly describes both changes made: adding z-index to MobileTreeDrawer sticky header and removing duplicate chevron from PageSize component, with before/after visual evidence provided.

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

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@danielroe danielroe added this pull request to the merge queue Feb 11, 2026
Merged via the queue into npmx-dev:main with commit da315a4 Feb 11, 2026
17 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