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

Favorites screen collection view crash. #3974

Closed
iccub opened this issue Jul 28, 2021 · 2 comments · Fixed by #3985
Closed

Favorites screen collection view crash. #3974

iccub opened this issue Jul 28, 2021 · 2 comments · Fixed by #3985

Comments

@iccub
Copy link
Contributor

iccub commented Jul 28, 2021

Description:

Steps to Reproduce

Hard to reproduce, but xcode has a lot of crashes reported.

Actual result:

Expected result:

Reproduces how often: [Easily reproduced, Intermittent Issue]

Brave Version:

Device details:

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@iccub iccub added this to the 1.30 milestone Jul 28, 2021
@iccub iccub changed the title Favorites collection view crash. Favorites screen collection view crash. Jul 28, 2021
@iccub iccub self-assigned this Jul 28, 2021
@iccub iccub modified the milestones: 1.30, 1.31 Aug 5, 2021
@bsclifton
Copy link
Member

Shared in PR, but please contact me once this is ready 😄 I can reproduce in Beta and Dev. I'd love to verify the fix

I basically reproduced by going to craigslist, adding a bunch of favorites (over time)... and then deleting them after some time. In the craigslist example, I usually delete the favorite when the person pulls the listing (which might be throwing a 500 error or similar)

@srirambv
Copy link
Contributor

Verification passed on iPhone XR with iOS 13.5 running 1.30(21.8.12.8)

  • Verified able to remove favourites when focus is on URL bar (show more mode)
  • Verified able to reorder favourites when focus is on URL bar (show more mode)
  • Verified able to edit favourites when focus is on URL bar (show more mode)
  • Verified able to open a favourite in new tab/private tab via context menu when focus is on URL bar (show more mode)
  • Verified able to open/edit/remove a favourite on an open page when focus is on URL bar (show more mode)
  • Verified favourites work correctly on both normal and private tab

Verification passed on iPhone 7+ with iOS 14.5.1 running 1.30(21.8.12.8)

  • Verified able to remove favourites when focus is on URL bar (show more mode)
  • Verified able to reorder favourites when focus is on URL bar (show more mode)
  • Verified able to edit favourites when focus is on URL bar (show more mode)
  • Verified able to open a favourite in new tab/private tab via context menu when focus is on URL bar (show more mode)
  • Verified able to open/edit/remove a favourite on an open page when focus is on URL bar (show more mode)
  • Verified favourites work correctly on both normal and private tab

Verification passed on iPad Pro with iOS 14.6 running 1.30(21.8.12.8)

  • Verified able to remove favourites when focus is on URL bar (show more mode)
  • Verified able to reorder favourites when focus is on URL bar (show more mode)
  • Verified able to edit favourites when focus is on URL bar (show more mode)
  • Verified able to open a favourite in new tab/private tab via context menu when focus is on URL bar (show more mode)
  • Verified able to open/edit/remove a favourite on an open page when focus is on URL bar (show more mode)
  • Verified favourites work correctly on both normal and private tab

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