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

Badge count animation between lion head and password manager needs to synchronize #8186

Closed
srirambv opened this issue Apr 10, 2017 · 6 comments
Assignees
Labels
design A design change, especially one which needs input from the design team. needs-info Another team member needs information from the PR/issue opener. QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified

Comments

@srirambv
Copy link
Collaborator

srirambv commented Apr 10, 2017

Test plan:

#9014 (comment)


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

  • Describe the issue you encountered:
    Badge count animation between lion head and password manager needs to synchronize

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

  • Brave Version (revision SHA):
    Brave 0.14.2
    rev 61ae1a7

  • Steps to reproduce:

    1. Enable LastPass with a stored credentials
    2. Visit the site for which the credentials is saved
    3. Hover over the next tab, LastPass badge is hidden, lion head badge still retained
    4. Switch back to the tab for which LastPass badge count is still not shown but lion head badge shows the badge
    5. Switch to the new tab, LastPass badge is still shown, lion head badge is hidden
  • Actual result:
    Badge count animation between lion head and password manager needs to synchronize

  • Expected result:
    Badge count should be shown properly on tabs

  • 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:
    badgecount

  • Any related issues:

@srirambv srirambv added design A design change, especially one which needs input from the design team. QA/test-plan-specified release/not-blocking labels Apr 10, 2017
@srirambv srirambv added this to the 0.14.3 milestone Apr 10, 2017
@cezaraugusto
Copy link
Contributor

@srirambv could you describe more fully what's the expected result here?

@cezaraugusto cezaraugusto added the needs-info Another team member needs information from the PR/issue opener. label Apr 25, 2017
@cezaraugusto
Copy link
Contributor

we could have an animation to smooth transitions but not sure if that's what you mean

@srirambv
Copy link
Collaborator Author

@cezaraugusto Yes there is no smooth transition in the counts. The badge count on lion head appears first and then for LastPass. Also when tab switches Lastpass retains badge and Lion head doesn't retain. On hover LastPass badge is lost but Lion head retians the badge

@bbondy bbondy modified the milestones: 0.15.3, 0.15.2 May 2, 2017
@cezaraugusto
Copy link
Contributor

cezaraugusto commented May 23, 2017

after a while animating badges I started to like them as-is if a subtle transition is included:

badge_animation_delay

cc @srirambv @bradleyrichter what do you both think?

cezaraugusto added a commit that referenced this issue May 23, 2017
@luixxiul
Copy link
Contributor

CC @bradleyrichter for design review

@srirambv
Copy link
Collaborator Author

Fade in effect on LastPass is a little slower compared to Honey. But much better than the initial transition 👍
8186_1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design A design change, especially one which needs input from the design team. needs-info Another team member needs information from the PR/issue opener. QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified
Projects
None yet
Development

No branches or pull requests

5 participants