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

The ETP shield icon is wrongly displayed when opening files in the New Tab/Window and permissions are denied #3282

Closed
Softvision-RemusDranca opened this issue Apr 29, 2020 · 1 comment · Fixed by #3287
Assignees
Labels
in progress [QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Milestone

Comments

@Softvision-RemusDranca
Copy link

[Affected Versions]:

  • Firefox Reality version 10 RC3 (5b346b6).

[Affected Platforms]:

  • Pico Neo 2 - Android - 8.1.0
  • Oculus Quest - Android 7.1.1
  • HTC Vive Focus - Android 7.1.2

[Prerequisites]:

  • The headset is turned on and the app is opened.
  • Have an available file in the Downloads library.
  • The Permissions to access External Storage is disabled.

[Steps to reproduce]:

  1. Click the Downloads button from the tray.
  2. Hover over a file and click the 3 dot menu button.
  3. Click the "Open in a Tab" button.
  4. Deny the permissions from the prompt displayed.
  5. Observe if the ETP shield icon is displayed.

[Expected results]:

  • The icon is not displayed.

[Actual results]:

  • The icon is displayed.

[Notes]:

  • This issue is also reproducible when opening files in a New Window.
  • Attached is a screen recording of the issue: link.
@Softvision-RemusDranca Softvision-RemusDranca added the [QA]:Normal issue Label for QA to mark normal issues logged label Apr 29, 2020
@keianhzo keianhzo self-assigned this Apr 29, 2020
@keianhzo keianhzo added this to the #10 features milestone Apr 29, 2020
bluemarvin pushed a commit that referenced this issue Apr 29, 2020
)

* Only show URL buttons for http/https uris

* Also hide URL bar icons if in library
@Softvision-RemusDranca
Copy link
Author

I have verified this issue and it's no longer reproducible with the Firefox Reality version 10 RC4 (2999129). The ETP shield icon is no longer displayed when opening files in the New Tab/Window and permissions are denied.

Tested on HTC Vive Focus (Android 7.1.2), Oculus Quest (Android 7.1.1), and Pico Neo 2 (Android - 8.1.0) devices.

@Softvision-RemusDranca Softvision-RemusDranca added the [QA]:Verified fixed Label for QA to mark verified fixed issues label May 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
in progress [QA]:Normal issue Label for QA to mark normal issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants