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

Exiting PB mode causes normal tab to become blank with incorrect URL #888

Closed
srirambv opened this issue Feb 11, 2019 · 4 comments
Closed

Comments

@srirambv
Copy link
Contributor

Description:

Exiting PB mode causes normal tab to become blank with incorrect URL

Steps to Reproduce

  1. Open any link in a new tab
  2. Long press on a link on the page and open in new tab
  3. Switch to the new tab by tapping on the background tab, do not use the switch button
  4. Open the same link or different link from step 2 in a new private tab
  5. Open tab container and exit private mode by clicking on the Private button on the bottom left
  6. Close the second tab from step 3 by swiping the tab to left/right in tab container
  7. Open the tab from step 1, URL bar has URL which was opened in a new tab/private tab, entire tab is blank but tab title is correct. URL uses the dark mode private tab colour for normal tab
  8. Reload the tab just changes the URL to correct value, page doesn't load
  9. Current tab is useless have to remove app from memory and relaunch to reuse the tab

Actual result:
iospb

Expected result:
Should switch back to normal tabs correctly

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Brave Version:
1.8(19.02.07.22)

Device details:
All devices

Website problems only:

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

Additional Information

@srirambv srirambv added bug priority/P1 A very extremely bad problem. We might push a hotfix for it. QA/Yes release/blocking private-browsing labels Feb 11, 2019
@srirambv srirambv added this to the 1.8 milestone Feb 11, 2019
@iccub
Copy link
Contributor

iccub commented Feb 11, 2019

Is this a regression?

@srirambv
Copy link
Contributor Author

No this is not a regression. Happens on Appstore build as well.

@iccub iccub self-assigned this Feb 11, 2019
@iccub iccub removed their assignment Feb 11, 2019
@srirambv srirambv modified the milestones: 1.8, 1.9 Feb 12, 2019
@srirambv
Copy link
Contributor Author

Not a regression so moving to 1.9

Mahmoud333 added a commit to Mahmoud333/brave-ios that referenced this issue Mar 24, 2019
Mahmoud333 added a commit to Mahmoud333/brave-ios that referenced this issue Mar 24, 2019
@srirambv srirambv removed this from the 1.9 milestone Mar 27, 2019
iccub pushed a commit to Mahmoud333/brave-ios that referenced this issue Aug 5, 2019
@iccub iccub added this to the 1.12 milestone Aug 5, 2019
@iccub iccub closed this as completed in 1771ba6 Aug 5, 2019
@srirambv
Copy link
Contributor Author

srirambv commented Sep 6, 2019

Verification passed on iPhone XR with iOS 13 running 1.12(19.09.05.20)

  • Verified steps from original issue description on a clean install
  • Verified that issue reproduced on 1.11.4 and not reproducible after upgrade to 1.12(19.09.05.20)

Verification PASSED on iPad Air 3rd Generation using iOS 13.1 running 1.12 (19.09.11.22):

  • Verified steps from original issue description on a clean install
  • Verified that issue reproduced on 1.11.4 and not reproducible after upgrade to 1.12 (19.09.11.22)

Verification passed on iPhone 7+ with iOS 12.4.1 running 1.12(19.09.13.06)

  • Verified steps from original issue description on a clean install
  • Verified that issue reproduced on 1.11.4 and not reproducible after upgrade to 1.12(19.09.05.20)

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

No branches or pull requests

4 participants