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

Add long press reload that bypasses cache #2922

Merged
merged 4 commits into from
Mar 12, 2020
Merged

Add long press reload that bypasses cache #2922

merged 4 commits into from
Mar 12, 2020

Conversation

bluemarvin
Copy link
Contributor

No description provided.

@bluemarvin bluemarvin added this to the #10 features milestone Mar 4, 2020
@bluemarvin bluemarvin self-assigned this Mar 4, 2020
@bluemarvin
Copy link
Contributor Author

@MortimerGoro @keianhzo I'm not convinced the developer switch is necessary. I would okay with removing it and just having long press reload. Also, there is a bypass proxy flag. Should we support that too?

@MortimerGoro
Copy link
Contributor

MortimerGoro commented Mar 4, 2020 •
edited
Loading

@bluemarvin I'd keep the developer switch, it can help with discoverability and if you have to test a page tons of times it's quicker to use normal click than long click gesture.

@bluemarvin
Copy link
Contributor Author

@MortimerGoro @keianhzo I think this is ready to land. I think we should go with out changing the color or tooltip for now. Opinions?

keianhzo
keianhzo suggested changes Mar 11, 2020
View reviewed changes
Copy link
Contributor

@keianhzo keianhzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In Pico devices (I guess related to Android 8) I get also the click event after the logn click.

@bluemarvin
Copy link
Contributor Author

In Pico devices (I guess related to Android 8) I get also the click event after the logn click.

Is there an Android 8 way to handle this?

@bluemarvin bluemarvin requested a review from keianhzo March 11, 2020 17:18
@bluemarvin
Copy link
Contributor Author

@keianhzo Okay, I think it is fixed.

@keianhzo keianhzo merged commit 81eb75b into master Mar 12, 2020
@keianhzo keianhzo deleted the long-press-reload branch March 12, 2020 17:04
@keianhzo keianhzo added the QA Attention QA label Mar 12, 2020
@Softvision-GeluHaiduc
Copy link

I’ve performed exploratory testing on this new feature using the Oculus Go, HTC Vive Focus Plus, Pico Neo2 and Pico G2 4K headsets, logged all the issues found and also written test cases to cover the new functionality.

@Softvision-GeluHaiduc Softvision-GeluHaiduc removed the QA Attention QA label Mar 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@keianhzo keianhzo keianhzo approved these changes

@MortimerGoro MortimerGoro Awaiting requested review from MortimerGoro

Assignees

@bluemarvin bluemarvin

Labels
None yet
Projects
None yet
Milestone
#10 features
Development

Successfully merging this pull request may close these issues.
4 participants
@bluemarvin @MortimerGoro @Softvision-GeluHaiduc @keianhzo