Skip to content

Add optional explainer links context to WPT coverage analysis#6077

Open
szy196 wants to merge 9 commits intomainfrom
add-explainer-links-ui
Open

Add optional explainer links context to WPT coverage analysis#6077
szy196 wants to merge 9 commits intomainfrom
add-explainer-links-ui

Conversation

@szy196
Copy link
Collaborator

@szy196 szy196 commented Mar 3, 2026

This change adds the ability for users to optionally include feature explainer links as context when requesting an AI-powered WPT coverage evaluation.

Key changes:

  • Added an "Include feature explainers" checkbox to the Prerequisites Checklist in the WPT evaluation page.
  • Updated the checklist UI to show success/danger/info icons based on the checkbox state and availability of links.
  • Standardized icon sizes to 20px across the checklist.
  • Added a "(no value)" placeholder when explainers are included but no links are provided.
  • Updated the "About" section to explain the optional explainer context.
  • Modified the client-side API to pass the include_explainer parameter.
  • Added comprehensive tests for the new UI behavior and API integration.

@szy196 szy196 force-pushed the add-explainer-links-ui branch from 63384a7 to 0a7110e Compare March 3, 2026 00:39
DanielRyanSmith and others added 7 commits March 3, 2026 22:58
This change adds the ability for users to optionally include feature
explainer links as context when requesting an AI-powered WPT coverage
evaluation.

Key changes:
- Added an "Include feature explainers" checkbox to the Prerequisites
  Checklist in the WPT evaluation page.
- Updated the checklist UI to show success/danger/info icons based on
  the checkbox state and availability of links.
- Standardized icon sizes to 20px across the checklist.
- Added a "(no value)" placeholder when explainers are included but
  no links are provided.
- Updated the "About" section to explain the optional explainer context.
- Modified the client-side API to pass the `include_explainer` parameter.
- Added comprehensive tests for the new UI behavior and API
  integration.
@szy196 szy196 force-pushed the add-explainer-links-ui branch from ddaec08 to 6714f1e Compare March 4, 2026 21:58
@szy196 szy196 force-pushed the add-explainer-links-ui branch from 5459bc6 to fe4f5aa Compare March 4, 2026 22:04
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.

2 participants