Skip to content

fix: match mobile search button style to desktop version#1989

Closed
ComputelessComputer wants to merge 1 commit intomainfrom
devin/1764390124-mobile-search-button-style
Closed

fix: match mobile search button style to desktop version#1989
ComputelessComputer wants to merge 1 commit intomainfrom
devin/1764390124-mobile-search-button-style

Conversation

@ComputelessComputer
Copy link
Collaborator

@ComputelessComputer ComputelessComputer commented Nov 29, 2025

Summary

Adds the missing hover:scale-[102%] class to the mobile "Go to Advanced Search" button in the SearchSection of the ai-notetaking page, making it consistent with the desktop version's hover behavior.

Before: Mobile button only had active:scale-[98%] (press effect) but no hover scale effect
After: Mobile button now has both hover:scale-[102%] and active:scale-[98%], matching desktop

Review & Testing Checklist for Human

  • View the ai-notetaking page on mobile viewport (< 640px) and verify the "Go to Advanced Search" button scales up slightly on hover
  • Confirm the hover effect matches the desktop button behavior

Notes

…style

Co-Authored-By: john@hyprnote.com <john@hyprnote.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link

netlify bot commented Nov 29, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 0f2b1b0
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/692a75dcd1d35f0008f28a68
😎 Deploy Preview https://deploy-preview-1989--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 29, 2025

📝 Walkthrough

Walkthrough

A hover scale effect (102%) is added to the mobile-visibility Link for the advanced search feature in the SearchSection component of the AI note-taking page. Existing styling and functionality remain preserved.

Changes

Cohort / File(s) Summary
UI Hover Effect Enhancement
apps/web/src/routes/_view/product/ai-notetaking.tsx
Added hover:scale-[102%] class to the advanced search Link in SearchSection, amplifying button size on hover while retaining existing shadow, scale, layout, colors, and transition properties

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • ui changes #1796 — Modifies the same SearchSection advanced-search Link appearance and hover/visibility behavior
  • landing #1649 — Changes UI styling in the same file and SearchSection component

Suggested reviewers

  • yujonglee

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a hover scale effect to the mobile search button to match desktop styling.
Description check ✅ Passed The description is directly related to the changeset, clearly explaining the addition of the hover:scale class to the mobile button and providing context and testing guidance.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devin/1764390124-mobile-search-button-style

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f7c8d66 and 0f2b1b0.

📒 Files selected for processing (1)
  • apps/web/src/routes/_view/product/ai-notetaking.tsx (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

**/*.{ts,tsx}: Avoid creating a bunch of types/interfaces if they are not shared. Especially for function props, just inline them instead.
Never do manual state management for form/mutation. Use useForm (from tanstack-form) and useQuery/useMutation (from tanstack-query) instead for 99% of cases. Avoid patterns like setError.
If there are many classNames with conditional logic, use cn (import from @hypr/utils). It is similar to clsx. Always pass an array and split by logical grouping.
Use motion/react instead of framer-motion.

Files:

  • apps/web/src/routes/_view/product/ai-notetaking.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: Redirect rules - hyprnote
  • GitHub Check: Header rules - hyprnote
  • GitHub Check: Pages changed - hyprnote
  • GitHub Check: ci (macos, macos-14)
  • GitHub Check: fmt
🔇 Additional comments (1)
apps/web/src/routes/_view/product/ai-notetaking.tsx (1)

1075-1075: LGTM! Mobile button now matches desktop styling.

The addition of hover:scale-[102%] successfully aligns the mobile search button with its desktop counterpart at line 1066. The change achieves the PR objective of style consistency.

Note: Hover effects don't typically trigger on touch devices, but the existing active:scale-[98%] provides appropriate feedback for mobile interactions, and having both states ensures consistency if the viewport is resized or accessed with a pointing device.


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

@netlify
Copy link

netlify bot commented Nov 29, 2025

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit 0f2b1b0
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/692a75dc4db88200084ff70c
😎 Deploy Preview https://deploy-preview-1989--hyprnote-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@argos-ci
Copy link

argos-ci bot commented Nov 29, 2025

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
web (Inspect) ⚠️ Changes detected (Review) 3 changed Nov 29, 2025, 4:29 AM

@ComputelessComputer ComputelessComputer deleted the devin/1764390124-mobile-search-button-style branch December 14, 2025 15:20
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.

1 participant