[feat]: Update `<pf-v6-popover>` 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
- css, html, typescript
- Bereich
- accessibility, design, frontend, web-dev
Rechercherichtung
Start with elements/pf-v5-popover/, .claude/ADVICE.md, and the existing tooltip implementation referenced by the Floating UI dependency. Compare the PatternFly website and the listed prior-art elements, then update elements/package.json, eslint.config.js, _snippets/, and framework-integration/ as needed. Done means the PFv6 component, demos, API and accessibility tests, reviews, changeset, and staging/pfv6 targeting checklist are complete.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Summary
Update `` to implement the PatternFly v6 Popover component.
Rich content popup triggered by click, with header, body, footer, and close button. Supports multiple positions and auto-positioning.
Prior Art
| Source | Element | Status |
|---|---|---|
| PFE v4 | `` | Exists |
| cem | `` | Exists |
| cockpit | `` | Exists |
| chickadee | -- | -- |
Requirements
- MUST implement PFv6 visual designs
- MUST provide end-user feature parity with `@patternfly/react-core` Popover
- 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
- Uses Floating UI for positioning (shared dep with tooltip)
- React `position`: 12 values (`top`, `top-start`, `top-end`, `bottom`, `bottom-start`, `bottom-end`, `left`, `left-start`, `left-end`, `right`, `right-start`, `right-end`, `auto`)
- React `headerContent`, `bodyContent`, `footerContent` - use named slots
- React `headerComponent`: `h1`-`h6` - heading level in header slot
- React `hasAutoWidth` - removes max-width constraint; map to `auto-width` attribute
- React `showClose` (default true) - whether to render close button; map to `no-close` attribute (opt-out)
- React `distance`, `minWidth`, `maxWidth` - CSS custom properties preferred
- React `flipBehavior` - Floating UI flip fallback positions
- React `alertSeverityVariant` - adds alert icon to header for status popovers; map to `alert` attribute
- Focus management: focus moves into popover on open; Escape closes and returns focus to trigger
- Trigger element: use `for` attribute or wrap trigger in slot
Checklist
Rename
- Delete `elements/pf-v5-popover/`
- 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
- 394
- Forks
- 107
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
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
patternfly/patternfly-elements#3158 · 1 zugewiesene Person ·
-
patternfly/patternfly-elements#3136 · 1 Kommentar · 1 zugewiesene Person ·
-
docs
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
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Crush Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
ElementsProject/cln-application#167 · 1 Kommentar · 1 Reaktion ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Quantco/pnpm-licenses#17 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100