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

Switch to chromeless mode when app opens from a servo:// link #24095

Merged
merged 1 commit into from Aug 29, 2019

Conversation

@paulrouget
Copy link
Contributor

paulrouget commented Aug 29, 2019

Fix #24077

There's an issue where, in chromeless mode, when clicking a link in Servo, Edge opens the link (as expected) but not in a new tab, in a new window.


This change is Reviewable

@paulrouget paulrouget force-pushed the paulrouget:chromeless branch from dbc286f to 5fe9a62 Aug 29, 2019
@jdm
Copy link
Member

jdm commented Aug 29, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

📌 Commit 5fe9a62 has been approved by jdm

@highfive highfive assigned jdm and unassigned ferjm Aug 29, 2019
@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

Testing commit 5fe9a62 with merge 49d0e38...

bors-servo added a commit that referenced this pull request Aug 29, 2019
Switch to chromeless mode when app opens from a servo:// link

Fix #24077

There's an issue where, in chromeless mode, when clicking a link in Servo, Edge opens the link (as expected) but not in a new tab, in a new window.

<!-- 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/24095)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Aug 29, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

Testing commit 5fe9a62 with merge 39bd455...

bors-servo added a commit that referenced this pull request Aug 29, 2019
Switch to chromeless mode when app opens from a servo:// link

Fix #24077

There's an issue where, in chromeless mode, when clicking a link in Servo, Edge opens the link (as expected) but not in a new tab, in a new window.

<!-- 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/24095)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 29, 2019

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

@bors-servo bors-servo merged commit 5fe9a62 into servo:master Aug 29, 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
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.

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