chore: Change navbar's search input icon size

[dougfabris]

Proposed changes (including videos or screenshots)

Change the icon size to be equal the clear button, avoiding navbar resizing

before

after

Issue(s)

Steps to test or reproduce

Further comments

CORE-2309

Summary by CodeRabbit

• Style
  • Increased the size of the search icon in the navigation bar for improved visibility.

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is targeting the wrong base branch. It should target 8.7.0, but it targets 8.6.0

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

⚠️ No Changeset found

Latest commit: 2842271

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

The magnifier icon rendered inside the TextInput addon in NavBarSearch.tsx has its size prop changed from x16 to x20 for the non-dirty (default) state. No logic, handlers, or other component behavior are modified.

Changes

NavBar Search Icon Size

Layer / File(s)	Summary
Search icon size apps/meteor/client/navbar/NavBarSearch/NavBarSearch.tsx	Magnifier icon size prop updated from x16 to x20 in the non-dirty TextInput addon.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

type: chore

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: adjusting the navbar search input icon size from x16 to x20.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)

• F20B-4791: Request failed with status code 401

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.15%. Comparing base (ffaa115) to head (2842271).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #41042      +/-   ##
===========================================
- Coverage    70.16%   70.15%   -0.02%     
===========================================
  Files         3361     3361              
  Lines       129690   129690              
  Branches     22490    22465      -25     
===========================================
- Hits         91000    90978      -22     
- Misses       35378    35398      +20     
- Partials      3312     3314       +2     

Flag	Coverage Δ
e2e	59.32% <ø> (-0.12%)	⬇️
e2e-api	47.12% <ø> (+0.76%)	⬆️
unit	70.06% <ø> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}