fix(perps): use one-line ListItemSelect for price grouping options

[brianacnguyen]

Description

Price grouping options in the Perps order book sheet used the default two-line ListItemSelect layout, which left empty vertical space under each value and made the list look sparse.

This PR switches those rows to the MMDS one-line variant so each option renders as a compact single-line row.

What changed:

• Import ListItemVariant in PerpsOrderBookView
• Set variant={ListItemVariant.OneLine} on price grouping ListItemSelect items

Changelog

CHANGELOG entry: null

Related issues

Fixes: N/A

Manual testing steps

Feature: Perps order book price grouping sheet density

  Scenario: user opens price grouping options
    Given the user has Perps enabled and opens an order book for a market
    And the order book depth/price grouping control is visible

    When the user opens the Select Price Grouping bottom sheet
    Then each grouping option (e.g. 1, 2, 5, 10, 100, 1000) renders as a compact one-line ListItemSelect
    And the currently selected grouping still shows the selected background and check icon
    And selecting a different option updates the grouping and closes or applies as before

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Performance checks (if applicable)

• I've tested on Android
  • Ideally on a mid-range device; emulator is acceptable
• I've tested with a power user scenario
  • Use these power-user SRPs to import wallets with many accounts and tokens
• I've instrumented key operations with Sentry traces for production performance metrics
  • See trace() for usage and addToken for an example

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

**Notes for you**
- No ticket was found on the branch — add an issue id to `Fixes:` (or use `Refs:`) before Ready for review
- Attach before/after screens of the price grouping sheet (you already have a before)
- Open as a draft: `gh pr create --draft` if you want that next

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Cosmetic MMDS variant change on a single bottom sheet; no logic, API, or data-handling impact.
> 
> **Overview**
> The Perps order book **Select Price Grouping** bottom sheet now uses the MMDS **one-line** `ListItemSelect` layout instead of the default two-line variant, so grouping values (e.g. 1, 10, 100) render as compact single-line rows without extra empty space.
> 
> **What changed:** `ListItemVariant` is imported from `@metamask/design-system-react-native`, and each depth-band option sets `variant={ListItemVariant.OneLine}`. Selection behavior, check icon, and grouping logic are unchanged.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 27c40ac6264c18627f7dd857d487b9c3ebe37448. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[metamask-ci]

PR template — items to address before "Ready for review"

Warnings — informational, address before merging:

• Pre-merge author checklist has unchecked items (e.g. "I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards."). Every box must be consciously checked — see docs/readme/ready-for-review.md.

See docs/readme/ready-for-review.md for the full Definition of Ready for Review.

[github-actions]

🔍 Smart E2E Test Selection

• Selected E2E tags: SmokePerps, SmokeWalletPlatform, SmokeConfirmations
• Selected Performance tags: None (no tests recommended)
• Risk Level: low
• AI Confidence: 85%

click to see 🤖 AI reasoning details

E2E Test Selection:
The PR changes only one file: PerpsOrderBookView.tsx. The change is minimal:

1. Imports ListItemVariant from the component library
2. Adds variant={ListItemVariant.OneLine} prop to a ListItemSelect component in the grouping selector of the order book view

This is a low-risk UI variant change affecting the Perps Order Book view's list item display style.

SmokePerps is selected because the change directly affects a Perps view component (PerpsOrderBookView).

SmokeWalletPlatform is selected per the SmokePerps tag description: "Perps is also a section inside the Trending tab (SmokeWalletPlatform); changes to Perps views (headers, lists, full views) affect Trending."

SmokeConfirmations is selected per the SmokePerps tag description: "When selecting SmokePerps, also select SmokeConfirmations (Add Funds deposits are on-chain transactions)."

The change is isolated to the Perps Order Book view with no impact on shared navigation, modals, or other app areas.

Performance Test Selection:
The change is a minor UI variant prop addition (ListItemVariant.OneLine) to a ListItemSelect component in the Perps Order Book view. This is a display/styling change that does not affect performance-sensitive operations like data fetching, rendering loops, or app startup. No performance tags are warranted.

View GitHub Actions results

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud