[feat]: Update `<pf-v6-modal>` for PatternFly v6
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- html, typescript
- Bereich
- accessibility, frontend
Rechercherichtung
Start with .claude/ADVICE.md and inspect elements/pf-v5-modal/ before planning the replacement. Check elements/package.json, eslint.config.js, _snippets/, and framework-integration/ for references to update. Done means a PatternFly v6 modal with the requested API, demos, public API and a11y tests, review checks, changeset, and a PR targeting staging/pfv6.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Summary
Update `` to implement the PatternFly v6 Modal component.
Dialog overlay for focused tasks, confirmations, or information display. Supports size variants and custom footers.
Prior Art
| Source | Element | Status |
|---|---|---|
| PFE v4 | `` | Exists |
| cem | `` | Exists |
| cockpit | `` | Exists |
| chickadee | `` | Exists |
Notes
- Should use the native `` element internally
- Use `popover` attribute where appropriate for light-dismiss behavior
Requirements
- MUST implement PFv6 visual designs
- MUST provide end-user feature parity with `@patternfly/react-core` Modal
- MAY adjust element API to leverage web platform strengths
- SHOULD maintain CSS custom property theming compatibility
- SHOULD expose useful CSS shadow parts
Element-specific considerations
- React `variant`: `small`, `medium`, `large`, `xlarge`, `default` - map to `size` attribute
- React `isScrollable` - scrollable body with sticky header/footer; map to `scrollable` attribute
- React `title`, `titleIconVariant` (`success`, `danger`, `warning`, `info`, `custom`) - use slot or attribute
- React `actions` - footer button group; use `slot="footer"`
- React `description` - subtitle below title
- React `hasNoBodyWrapper` - removes the default `ModalBoxBody` padding wrapper
- React `disableFocusTrap` - should almost never be needed; do not expose unless truly necessary
- Native `` handles: focus trap, Escape key close, backdrop (`::backdrop` pseudo-element)
- Light-dismiss (click outside): use `dialog.addEventListener("click", ...)` pattern on backdrop
- `showModal()` vs `show()` vs `popover`: use `showModal()` for proper modal semantics (blocks interaction)
- Expose `open` property/attribute for declarative open/close
Checklist
Rename
- Delete `elements/pf-v5-modal/`
- Update `elements/package.json` exports
- Update `eslint.config.js` ignore globs if applicable
- Update docs `_snippets/` and `framework-integration/` references
Implementation
- API per `.claude/ADVICE.md`
- CSS uses v6 tokens, logical properties, nesting
- `@summary` and JSDoc on element class
Demos
- Demos match patternfly.org naming/structure
- CSS custom properties match computed styles from patternfly.org (Chrome MCP)
- Visual parity verified via Chrome MCP
Tests
- Public API covered (attributes, properties, events, slots)
- `a11ySnapshot` assertions
Reviews
- `/review-api` passes
- `/review-demos` passes
- `/review-a11y` passes
- `eslint` and `stylelint` pass
Ship
- Add changeset
- PR targets `staging/pfv6` (NOT `main`)
- Vorherrschende Sprache
- TypeScript
- Sterne
- 395
- Forks
- 107
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus patternfly/patternfly-elements
-
[bug] ssr-shims.js uses deprecated globalThis.litSsrCallConnectedCallbackEvtl. wieder frei @zeroedin hat das vor 60 Tagen übernommen, und es ist kein Pull Request offen. Offenbug
patternfly/patternfly-elements#3158 · 1 zugewiesene Person ·
-
chore: evaluate removing isolatedDeclarations from tsconfigEvtl. wieder frei @bennypowers hat das vor 150 Tagen übernommen, und es ist kein Pull Request offen. Offen
patternfly/patternfly-elements#3136 · 1 Kommentar · 1 zugewiesene Person ·
-
chore: update docs scaffolding for v6 elementsEvtl. wieder frei @zeroedin hat das vor 150 Tagen übernommen, und es ist kein Pull Request offen. Offendocs
patternfly/patternfly-elements#3131 · 1 zugewiesene Person ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
patternfly/patternfly-elements#3122 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 38/100
patternfly/patternfly-elements#3118 ·
Alle Issues in patternfly/patternfly-elements
Ähnliche Issues
-
area/dashboard kind/bug QA/dev-automation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
rancher/dashboard#19379 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 5 Tagen
-
perf(core): getComments() runs the approved count and the comment list as two sequential queriesOffenarea/core bot:bug bot:working
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
emdash-cms/emdash#3905 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
lingdojo/kana-dojo#31728 · 1 Kommentar · 5 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
selective-claw: freshTailTurns=0 keeps ALL turns verbatim and summarizes none (slice(-0) === slice(0))Evtl. vergeben @zjncs hat das heute übernommen. Offencomponent:tokenless
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
agentic-os-org/ANOLISA#6112 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
rjsf-team/react-jsonschema-form#5439 ·
Maintainer antworten meist innerhalb von 1 Tag