Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Fix #3044: Search Suggestion Text Color blinks when text is entered #3091

Merged
merged 4 commits into from
Dec 2, 2020

Conversation

soner-yuksel
Copy link
Contributor

@soner-yuksel soner-yuksel commented Nov 27, 2020

This PR fixes Search suggestion Text Color flickering. Re-organization will be another PR.

Summary of Changes

This pull request fixes #3044

Submitter Checklist:

  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()

Test Plan:

  • Load Brave Browser
  • Go Settings - Search Engines - Enable Search Suggestions
  • Create a new Tab and load a Webpage and press search bar
  • Start typing something
  • Observe Search Suggestion Boxes Text color is not flickering from blue to white

Screenshots:

ezgif-7-f5203d6961b3

Reviewer Checklist:

  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate unit test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable).
  • Issue is assigned to a milestone (should happen at merge time).

Copy link
Contributor

@iccub iccub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good overall, I found one artifact though, happens on iOS 12, see top left in the search suggestions view, there's something blue there

Zrzut ekranu 2020-11-30 o 15 07 21

@soner-yuksel
Copy link
Contributor Author

Code looks good overall, I found one artifact though, happens on iOS 12, see top left in the search suggestions view, there's something blue there

Yeah what is that artifact let me check why it might happen.

@soner-yuksel soner-yuksel force-pushed the search-suggestion-fix branch from 743d256 to c8a009a Compare December 1, 2020 16:49
@soner-yuksel
Copy link
Contributor Author

Code looks good overall, I found one artifact though, happens on iOS 12, see top left in the search suggestions view, there's something blue there

iOS 12 Search Icon Artifact Problem is fixed.

@soner-yuksel soner-yuksel requested a review from iccub December 1, 2020 16:52
Copy link
Contributor

@iccub iccub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, tested on iOS 14 and 12

@soner-yuksel soner-yuksel requested a review from iccub December 1, 2020 22:09
@soner-yuksel soner-yuksel merged commit f2fa861 into development Dec 2, 2020
@soner-yuksel soner-yuksel deleted the search-suggestion-fix branch December 2, 2020 18:26
@soner-yuksel soner-yuksel added this to the 1.23 milestone Dec 2, 2020
@soner-yuksel soner-yuksel self-assigned this Dec 4, 2020
@kylehickinson kylehickinson removed this from the 1.23 milestone Feb 1, 2021
@kjozwiak kjozwiak added this to the 1.23 milestone Feb 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Search suggestion text color blinks when text is entered
4 participants