Skip to content

Global Search Refactor - Fix Input Border

Zack Cuddy requested to merge 345745_06-better-input-border into master

What does this MR do and why?

This addresses a regression found in here: !74244 (comment 731485313)
Work towards #297396 (closed)

This change is behind a feature flag (:new_header_search)
Feature Flag Rollout Issue: #339348 (closed)

This change fixes the issue of the inset box shadow (visually looks to be a border) that is new to the Header Search. This change focuses on making the box shadow match better with all the themes using similar logic to the input backgrounds.

Screenshots or screen recordings

Zoom in of the change
Difference_Up_Close

Before After
Indigo Indigo_Before Indigo_After
Light Indigo Light_Indigo_Before Light_Indigo_After
Blue Blue_Before Blue_After
Light Blue Light_Blue_Before Light_Blue_After
Green Green_Before Green_After
Light Green Light_Green_Before Light_Green_After
Red Red_Before Red_After
Light Red Light_Red_Before Light_Red_After
Dark Dark_Before Dark_After
Light Light_Before Light_After
Dark Mode Dark_Mode_Before Dark_Mode_After

How to set up and validate locally

  1. Fetch and checkout this branch
  2. Activate the Feature Flag
  3. Cycle through the themes and see the new inset box shadow

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Zack Cuddy

Merge request reports

Loading