Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

allow search icon to show while loading #6532

Conversation

kaizensoze
Copy link
Contributor

Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

There are a bunch of tests failing unrelated to this change:

  1. lets you pin a tile (and shows the pinned icon afterwards)
  2. detects blocked elements in private tab
  3. "after each" hook for "detects adblock resources"
    ...

Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes brave#6518.
@bradleyrichter
Copy link
Contributor

thanks! I think @bsclifton will be thrilled to review it.

@gyandeeps
Copy link
Contributor

Related PR: #6319
I hope both changes play nice together.

@luixxiul luixxiul added this to the 0.13.1 milestone Jan 14, 2017
@luixxiul luixxiul changed the base branch from master to 0.13.1-branch January 14, 2017 14:55
@bsclifton bsclifton force-pushed the 0.13.1-branch branch 2 times, most recently from 0da267f to 7f84176 Compare January 17, 2017 21:59
Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

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

++ awesome, thanks!

@cezaraugusto cezaraugusto merged commit 2c4b720 into brave:0.13.1-branch Jan 18, 2017
bsclifton pushed a commit that referenced this pull request Jan 18, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
bsclifton pushed a commit that referenced this pull request Jan 18, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
bsclifton pushed a commit that referenced this pull request Jan 20, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
NejcZdovc pushed a commit to NejcZdovc/browser-laptop that referenced this pull request Jan 23, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes brave#6518.
bsclifton pushed a commit that referenced this pull request Jan 23, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
bsclifton pushed a commit that referenced this pull request Jan 24, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
bsclifton pushed a commit that referenced this pull request Jan 25, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
bsclifton pushed a commit that referenced this pull request Jan 25, 2017
Allow the search icon to show while loading. This removes a display lag
time for URL bar icons.

Fixes #6518.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants