Skip to content

HTML: Remove UA styles for h1 in article, aside, nav, section #51673

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.

Sign up for GitHub

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

HTML: Remove UA styles for h1 in article, aside, nav, section #51673

wants to merge 1 commit into from
+0 −153

Conversation

zcorpan
Copy link
Member

@zcorpan zcorpan commented Mar 27, 2025

@zcorpan zcorpan mentioned this pull request Mar 27, 2025
Open
6 tasks
@community-tc-integration Community-TC Integration
Copy link

Uh oh! Looks like an error!

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/HOtbN2fyQL-q3YREDRaYvQ/bJq9nV95QzCyXpy-_-98zw",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/HOtbN2fyQL-q3YREDRaYvQ"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/HOtbN2fyQL-q3YREDRaYvQ/bJq9nV95QzCyXpy-_-98zw",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/HOtbN2fyQL-q3YREDRaYvQ"
      ]
    }
  ]
}
  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2025-03-28T08:02:36.440Z

@zcorpan zcorpan closed this Mar 28, 2025
@zcorpan zcorpan reopened this Mar 28, 2025
annevk added a commit to annevk/WebKit that referenced this pull request May 12, 2025
@annevk
Remove UA styles for h1 in article, aside, nav, and section
df7e564 
https://bugs.webkit.org/show_bug.cgi?id=292765

Reviewed by NOBODY (OOPS!).

Firefox has rolled this out to 5% of its release audience as of 138,
100% of the beta audience, and plans to ship this in 140 which is now
riding the trains.

Chrome has also taken steps in deprecating this pattern and plans to
fast follow Firefox.

I think we should do the same.

For testing I removed a redundant local test and imported a PR that
will be merged once the specification change goes in:
web-platform-tests/wpt#51673
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@domenic domenic domenic approved these changes

@annevk annevk Awaiting requested review from annevk

@foolip foolip Awaiting requested review from foolip

@jdm jdm Awaiting requested review from jdm

@jgraham jgraham Awaiting requested review from jgraham

Assignees

@foolip foolip

Labels
html status:needs-spec-decision
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zcorpan @domenic @foolip @wpt-pr-bot