Skip to content

/ servo

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

Jump to bottom

Incorrect serialization of wildcard-namespace type selectors when there is a default namespace #16020

Closed
bzbarsky opened this issue Mar 18, 2017 · 1 comment
Closed

Incorrect serialization of wildcard-namespace type selectors when there is a default namespace #16020

bzbarsky opened this issue Mar 18, 2017 · 1 comment
Assignees
@SimonSapin
Labels
A-content/css A-stylo C-has-manual-testcase

Comments

@bzbarsky
Copy link
Contributor

@bzbarsky bzbarsky commented Mar 18, 2017

Testcase (in stylo; I'm not sure how to get at this in servo proper):

<!DOCTYPE html>
<style>
  @namespace url(foo);
  *|a {}
</style>
<script>
  alert(document.styleSheets[0].cssRules[1].selectorText);
</script>

This alerts a which is not the same selector as *|a in this case.
@bholley bholley added the A-stylo label Mar 26, 2017
@highfive
Copy link

@highfive highfive commented Mar 26, 2017

cc @emilio
@bzbarsky bzbarsky mentioned this issue Apr 9, 2017
Closed
@SimonSapin SimonSapin self-assigned this May 15, 2017
SimonSapin added a commit that referenced this issue May 16, 2017
@SimonSapin
Fix serialization of namespace and universal selectors
Loading status checks…
390e688 
Fix #16017
Fix #16020
@SimonSapin SimonSapin mentioned this issue May 16, 2017
Merged
bors-servo added a commit that referenced this issue May 16, 2017
@bors-servo
Auto merge of #16890 - servo:qname, r=<try>
Loading status checks…
77aa36f 
Fix serialization of namespace and universal selectors

Fix #16017
Fix #16020

<!-- 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/16890)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue May 16, 2017
@bors-servo
Auto merge of #16890 - servo:qname, r=emilio
Loading status checks…
825a623 
Fix serialization of namespace and universal selectors

Fix #16017
Fix #16020

<!-- 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/16890)
<!-- Reviewable:end -->
emilio added a commit to emilio/servo that referenced this issue May 16, 2017
@SimonSapin @emilio
Fix serialization of namespace and universal selectors
Unverified
This user has not uploaded their public key yet.
GPG key ID: 056B727BB9C1027C Learn about signing commits
0689652 
Fix servo#16017
Fix servo#16020
bors-servo added a commit that referenced this issue May 16, 2017
@bors-servo
Auto merge of #16890 - servo:qname, r=emilio
Loading status checks…
a038128 
Fix serialization of namespace and universal selectors

Fix #16017
Fix #16020

<!-- 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/16890)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue May 17, 2017
@bors-servo
Auto merge of #16890 - servo:qname, r=emilio
Loading status checks…
c962458 
Fix serialization of namespace and universal selectors

Fix #16017
Fix #16020

<!-- 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/16890)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SimonSapin SimonSapin

Labels
A-content/css A-stylo C-has-manual-testcase
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@jdm @SimonSapin @bholley @bzbarsky @highfive
You can’t perform that action at this time.