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

Fix issue where pinned tab can't be really unpinned #10241

Closed
darkdh opened this issue Aug 1, 2017 · 1 comment
Closed

Fix issue where pinned tab can't be really unpinned #10241

darkdh opened this issue Aug 1, 2017 · 1 comment

Comments

@darkdh
Copy link
Member

darkdh commented Aug 1, 2017

Test Plan

  1. Go to feedly.com
  2. Pin the tab
  3. Bookmark it and change the location to https://feedly.com
  4. Unpin the pinned tab
  5. Relaunch Brave
  6. There shouldn't be any pinned tab

Original issue description

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

  • Describe the issue you encountered:
    Pinned tabs reappear next load after unpinning

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

  • Brave Version (revision SHA):
    526dc49

  • Steps to reproduce:

    1. go to feedly.com
    2. pin the tab
    3. bookmark it and change the location to https://feedly.com
    4. unpin the pinned tab
    5. relaunch Brave
  • Actual result:
    There shouldn't be any pinned feedly tab

  • Expected result:
    The tab is still there

  • 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?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:
    Pinned tabs reappear next load after unpinning #8477

cc @bsclifton

@darkdh darkdh self-assigned this Aug 1, 2017
@bsclifton bsclifton added this to the 0.18.x Hotfix milestone Aug 1, 2017
darkdh added a commit that referenced this issue Aug 3, 2017
also normalize site key without trailing slash

fix #10241

Auditors: @bsclifton, @bbondy

Test Plan:
1. Go to feedly.com
2. Pin the tab
3. Bookmark it and change the location to https://feedly.com
4. Unpin the pinned tab
5. Relaunch Brave
6. There shouldn't be any pinned tab
darkdh added a commit that referenced this issue Aug 3, 2017
also normalize site key without trailing slash

fix #10241

Auditors: @bsclifton, @bbondy

Test Plan:
1. Go to feedly.com
2. Pin the tab
3. Bookmark it and change the location to https://feedly.com
4. Unpin the pinned tab
5. Relaunch Brave
6. There shouldn't be any pinned tab
darkdh added a commit that referenced this issue Aug 3, 2017
also normalize site key without trailing slash

fix #10241

Auditors: @bsclifton, @bbondy

Test Plan:
1. Go to feedly.com
2. Pin the tab
3. Bookmark it and change the location to https://feedly.com
4. Unpin the pinned tab
5. Relaunch Brave
6. There shouldn't be any pinned tab
bsclifton pushed a commit that referenced this issue Aug 7, 2017
also normalize site key without trailing slash

fix #10241

Auditors: @bsclifton, @bbondy

Test Plan:
1. Go to feedly.com
2. Pin the tab
3. Bookmark it and change the location to https://feedly.com
4. Unpin the pinned tab
5. Relaunch Brave
6. There shouldn't be any pinned tab
@bsclifton bsclifton changed the title pinned tab can't be really unpinned Fix issue where pinned tab can't be really unpinned Aug 7, 2017
@bsclifton
Copy link
Member

Fixed with #10326

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.