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

Support Scroll in HoloLens mode #23831

Merged
merged 1 commit into from Jul 24, 2019
Merged

Support Scroll in HoloLens mode #23831

merged 1 commit into from Jul 24, 2019

Conversation

@paulrouget
Copy link
Contributor

paulrouget commented Jul 23, 2019

This change is Reviewable

@paulrouget
Copy link
Contributor Author

paulrouget commented Jul 23, 2019

@paulrouget
Copy link
Contributor Author

paulrouget commented Jul 24, 2019

This doesn't work in non-HoloLens env. I could use wheelchanged but I'm still talking to MS about how to properly handle gestures.

This approach is the "Right way" for hololens.

fix #23678

@jdm
Copy link
Member

jdm commented Jul 24, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

📌 Commit 380e54b has been approved by jdm

@highfive highfive assigned jdm and unassigned ferjm Jul 24, 2019
@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

Testing commit 380e54b with merge 46ff52c...

bors-servo added a commit that referenced this pull request Jul 24, 2019
Support Scroll in HoloLens mode

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23831)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

💔 Test failed - status-taskcluster

@CYBAI
Copy link
Collaborator

CYBAI commented Jul 24, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

Testing commit 380e54b with merge 2c1f120...

bors-servo added a commit that referenced this pull request Jul 24, 2019
Support Scroll in HoloLens mode

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23831)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

💔 Test failed - linux-rel-css

@jdm
Copy link
Member

jdm commented Jul 24, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

Testing commit 380e54b with merge b8af92a...

bors-servo added a commit that referenced this pull request Jul 24, 2019
Support Scroll in HoloLens mode

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23831)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: jdm
Pushing b8af92a to master...

@bors-servo bors-servo merged commit 380e54b into servo:master Jul 24, 2019
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
bors-servo added a commit that referenced this pull request Jul 25, 2019
[hololens] More events, remove most of the statics, better shutdown

Depends on #23831

- Buttons are greyed out when necessary
- Reload, Stop, Back and Fwd are properly bound
- Servo.cpp rewritten in a much cleaner way
- Removed most of the statics
- Improved (but not full) shutdown

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23845)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.