Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sharesheet menu is broken, Copy Clean Link and Report a Broken Site can't be added to the Favorites #36506

Closed
hffvld opened this issue Mar 1, 2024 · 2 comments · Fixed by brave/brave-core#22438 or brave/brave-core#22452
Assignees
Labels
bug OS/iOS Fixes related to iOS browser functionality priority/P1 A very extremely bad problem. We might push a hotfix for it. QA Pass - iPhone QA/Yes release-notes/exclude

Comments

@hffvld
Copy link
Contributor

hffvld commented Mar 1, 2024

Description:

Follow-up to brave/brave-ios#8070

Steps to Reproduce

  1. Launch Brave 1.63 or newer
  2. Go to any website
  3. Tap the Share button > Edit Actions...
  4. Add Copy Clean Link to the Favorites > Done > Observe Copy Clean Link is not shown in the Favorites
  5. Tap Edit Actions... > Observe Copy Clean Link is not shown anymore
  6. Add Report a Broken Site > Done > Observe Report a Broken Site is not added to the Favorites

Actual result:

Observed that:

  • Copy Clean Link disappeared from the list when trying to add it to the Favorites
  • Other items like Report a Broken Site can't be added to the Favorites

There is no option to restore the settings and see Copy Clean Link again. I checked Chrome/Safari and could add any item to the Favorites in both browsers and no items were hidden.

Copy Clean Link

2024-02-29_16-42-57.mp4

Report a Broken Site

2024-02-29_15-50-56.mp4

Expected result:

Copy Clean Link must be available in the list on Sharesheet and can be added to the Favorites.

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Brave build: 1.63 (161)
BraveCore: 1.63.161 (122.0.6261.57)
  • Can you reproduce this issue with the most recent build from TestFlight? YES
  • Can you reproduce this issue with the previous version of the current build from TestFlight? YES
  • Can you reproduce this issue with the current build from AppStore? YES

Device details:

- iPhone 14 / iOS 17.4 (21E217)
- iPad Air / iPadOS 16.7.2

Website problems only:

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

Additional Information

cc @soner-yuksel

@hffvld hffvld added bug QA/Yes OS/iOS Fixes related to iOS browser functionality labels Mar 1, 2024
@hffvld hffvld changed the title [Follow up to #8070] Sharesheet menu is broken, Clean Copy Link and Report a Broken Site can't be added to the Favorites [Follow up to #8070] Sharesheet menu is broken, Copy Clean Link and Report a Broken Site can't be added to the Favorites Mar 1, 2024
@hffvld hffvld added the priority/P1 A very extremely bad problem. We might push a hotfix for it. label Mar 1, 2024
@soner-yuksel soner-yuksel transferred this issue from brave/brave-ios Mar 1, 2024
@soner-yuksel soner-yuksel changed the title [Follow up to #8070] Sharesheet menu is broken, Copy Clean Link and Report a Broken Site can't be added to the Favorites Sharesheet menu is broken, Copy Clean Link and Report a Broken Site can't be added to the Favorites Mar 1, 2024
@brave-builds brave-builds added this to the 1.65.x - Nightly milestone Mar 4, 2024
@kjozwiak
Copy link
Member

kjozwiak commented Mar 5, 2024

The above requires 1.63.169 or higher for 1.63.x verification 👍

@soner-yuksel soner-yuksel self-assigned this Mar 6, 2024
@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Mar 7, 2024
@hffvld
Copy link
Contributor Author

hffvld commented Mar 7, 2024

Verified on iPhone 14 using version(s):

Device/OS: iPhone 14 / iOS 17.4
Brave build: 1.63 (169)
BraveCore: 1.63.169 (122.0.6261.111)

Fresh profile

STEPS:

  1. Follow the steps from Sharesheet menu is broken, Copy Clean Link and Report a Broken Site can't be added to the Favorites #36506 (comment)
  2. Verify

ACTUAL RESULTS:

  • Verified that Copy Clean Link is shown in the sharesheet menu
  • Verified that Copy Clean Link can be added to the Favorites and is not removed from the list
  • Verified that Copy Clean Link is working as expected
  • Verified that all Brave items from the list can be added to the Favorites

1 2
1 2
1 2
1 2
1 2
Clean.link.mp4
Upgraded profile

STEPS:

  1. Install and launch Brave 1.62.1
  2. Go to any website > Tap the Share button > Edit Actions... > Confirm Copy Clean Link is not shown here
  3. Add Report a Broken Site to the Favorites
  4. Done > Observe Report a Broken Site is not added to the Favorites
  5. Upgrade Brave to 1.63 (169) or newer build
  6. Tap the Share button > Verify that Copy Clean Link is shown here
  7. Edit Actions... > Add Copy Clean Link and Report a Broken Site to the Favorites
  8. Done > Verify Copy Clean Link is not disappearing anymore and any Brave item, can be added to the Favorites

2024-03-07_12-21-00.mp4

@hffvld hffvld added QA Pass - iPhone and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug OS/iOS Fixes related to iOS browser functionality priority/P1 A very extremely bad problem. We might push a hotfix for it. QA Pass - iPhone QA/Yes release-notes/exclude
Projects
None yet
4 participants