Skip to content

feat: mobile mini-cart dropdown tap + per-unit max for responsive range#4512

Merged
HardeepAsrani merged 1 commit into
developmentfrom
feat/issue-3161
Jun 15, 2026
Merged

feat: mobile mini-cart dropdown tap + per-unit max for responsive range#4512
HardeepAsrani merged 1 commit into
developmentfrom
feat/issue-3161

Conversation

@HardeepAsrani

@HardeepAsrani HardeepAsrani commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

Pro PR: #4512

Addresses https://github.com/Codeinwp/neve-pro-addon/issues/3159 (mini-cart dropdown on mobile tap) and https://github.com/Codeinwp/neve-pro-addon/issues/3161(width support for the header search form).

Search form width (Pro + theme)

The header search form had no width control, so it always rendered at its intrinsic width and couldn't be stretched to dominate the header.

  • Added a responsive Width control (field_width, px / %) on the Advanced Search Form.
  • Added a Stretch to fill available space toggle (flex_grow).

Mobile mini-cart dropdown (theme)

On mobile the dropdown mini-cart was hidden and tapping the cart icon just navigated to the cart page. Now, below the 960px breakpoint, tapping toggles the dropdown preview so customers can review the cart and keep shopping.

Will affect visual aspect of the product

YES/NO

Screenshots

Test instructions

Requires Neve + Neve Pro + WooCommerce active. Hard-refresh / clear cache before testing so rebuilt assets load.

A. Search width (#3161) — Customize → Header → add Advanced Search Form → Style tab

  • Controls present: new Width slider (with % / px unit buttons) and Stretch to fill available space toggle.
  • Width px: switch to px, set ~600 → field widens live; slider allows up to 1200.
  • Width %: switch to % → slider caps at 100; switching px→% clamps an out-of-range value (e.g. 600 → 100).
  • Per-device: set different values on desktop/tablet/mobile tabs → each applies at its breakpoint.
  • Stretch: toggle on (preview reloads) → field expands to fill its row.

B. Mobile mini-cart dropdown (#3159) — Cart Icon in header

  • Tap to open: tap the cart icon → dropdown preview opens, page does not navigate.
  • Tap to close: tap the icon again → closes.
  • Outside tap: open, then tap outside the dropdown → closes.
    I- nner actions: View cart / Checkout links navigate; removing a line item works via AJAX and the dropdown stays open.
  • Empty cart: empty cart → tapping the icon follows the default link to the cart page (no empty popup).
  • Desktop ≥960px: hover still opens the dropdown; clicking the icon still goes to the cart page (tap logic doesn't interfere).

Check before Pull Request is ready:

@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jun 10, 2026
@pirate-bot

pirate-bot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Plugin build for 7681c7d is ready 🛎️!

@HardeepAsrani HardeepAsrani added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jun 10, 2026
@HardeepAsrani HardeepAsrani force-pushed the feat/issue-3161 branch 2 times, most recently from 4a14a30 to fbebb0a Compare June 11, 2026 14:28
@HardeepAsrani HardeepAsrani requested a review from abaicus June 11, 2026 15:37
@rodica-andronache

Copy link
Copy Markdown
Contributor

@HardeepAsrani, regarding this https://github.com/Codeinwp/neve-pro-addon/issues/3159, opening/closing the cart on mobile is working well.
However, I thought it was worth sharing that the open cart is currently not that good-looking. If possible, I think, on mobile, it should show centered so you can see the entire cart
image

@HardeepAsrani HardeepAsrani merged commit c77650d into development Jun 15, 2026
16 of 23 checks passed
@HardeepAsrani HardeepAsrani deleted the feat/issue-3161 branch June 15, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants