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

Hide lion badge when block count is 0 #7873

Closed
srirambv opened this issue Mar 24, 2017 · 0 comments
Closed

Hide lion badge when block count is 0 #7873

srirambv opened this issue Mar 24, 2017 · 0 comments
Assignees
Labels
Milestone

Comments

@srirambv
Copy link
Collaborator

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Hide lion badge when adcount is 0

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    All

  • Brave Version (revision SHA):
    Brave 0.14.0
    rev ffa6f8f

  • Steps to reproduce:

    1. Do a clean install of 0.14.0 Preview 5
    2. Visit https://brave.com
    3. Lion badge shows count as 0,
  • Actual result:
    Lion badge shows count as 0

  • Expected result:
    Should not show badge when count is 0. Should only shown when (block count)>=1

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    N/A

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    image

  • Any related issues:
    cc: @NejcZdovc

@srirambv srirambv added design A design change, especially one which needs input from the design team. QA/test-plan-specified labels Mar 24, 2017
@srirambv srirambv added this to the 0.14.0 milestone Mar 24, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 24, 2017
Resolves brave#7873

Auditors: @bsclifton @luixxiul

Test Plan:
- go to brave.com
- badge should be hidden
@srirambv srirambv changed the title Hide lion badge when adcount is 0 Hide lion badge when block count is 0 Mar 25, 2017
bsclifton added a commit that referenced this issue Mar 25, 2017
bsclifton pushed a commit that referenced this issue Mar 25, 2017
Resolves #7873

Auditors: @bsclifton @luixxiul

Test Plan:
- go to brave.com
- badge should be hidden
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants