Skip to content

Add link to C-S-S core tests in index.html#408

Merged
jonathanKingston merged 1 commit intomainfrom
jonathanKingston-patch-2
Feb 24, 2026
Merged

Add link to C-S-S core tests in index.html#408
jonathanKingston merged 1 commit intomainfrom
jonathanKingston-patch-2

Conversation

@jonathanKingston
Copy link
Collaborator

@jonathanKingston jonathanKingston commented Feb 24, 2026

Note

Low Risk
Single static HTML link addition with no behavior, data, or security logic changes.

Overview
Adds a new entry to the home page (index.html) linking to ./content-scope-scripts/ so reviewers can directly access the C-S-S core test suite from the main navigation list.

Written by Cursor Bugbot for commit cbf890b. This will update automatically on new commits. Configure here.

@jonathanKingston jonathanKingston merged commit 0f473a1 into main Feb 24, 2026
2 checks passed
@jonathanKingston jonathanKingston deleted the jonathanKingston-patch-2 branch February 24, 2026 14:08
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

<li><a href='./windows-browser/script-injection/secureWorld_index.html'>Windows Browser: SecureWorld Script injection tests</a></li>
<li><a href='./content-scope-scripts/webcompat/'>Webcompat tests</a></li>
<li><a href='./content-scope-scripts/infra/'>C-S-S infra</a></li>
<li><a href='./content-scope-scripts/'>C-S-S core tests</a></li>
Copy link

Choose a reason for hiding this comment

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

Broken link to core tests

Low Severity

The new href points to ./content-scope-scripts/, but that path does not contain a routable landing page like content-scope-scripts/index.html. In static hosting, this makes the “C-S-S core tests” entry navigate to a missing page instead of a test index.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant