Skip to content

[ENG-46314] feat(webkit): add Badge (content)#663

Merged
robsongajunior merged 4 commits into
devfrom
feat/ENG-46314-badge
Jun 25, 2026
Merged

[ENG-46314] feat(webkit): add Badge (content)#663
robsongajunior merged 4 commits into
devfrom
feat/ENG-46314-badge

Conversation

@HerbertJulio

Copy link
Copy Markdown
Contributor

Summary

Adds the Badge content indicator, rebuilt on top of dev in the current repo format. Supersedes #660.

  • Severities: primary / secondary / success / warning / danger / default; sizes small/medium/large; non-interactive.
  • Story under apps/storybook/src/stories/components/content/badge/, title Components/Content/Badge.
  • Direct export @aziontech/webkit/badge (no category prefix).

Review feedback applied

  • CodeQL ([ENG-46314] feat(webkit): add Badge (content) #660, js/bad-tag-filter): the source.transform SFC-detector regex now uses the /i flag (/<script[\s>]/i), clearing the 'does not match upper case <SCRIPT>' alert. (This regex is a dev-time docs helper, not an HTML sanitizer — no security surface — but the /i flag makes the check green at zero cost.)

Notes

Replaces #660, which was branched from main (pre-dev-reorg). Diff is only the component's own files; merges cleanly into dev.

Compact severity-coded count/short-value indicator (primary/secondary/success/
warning/danger/default; small/medium/large). Story under stories/components,
direct `@aziontech/webkit/badge` export. The 'Show code' SFC detector regex is
case-insensitive (/i) so CodeQL's bad-tag-filter alert is resolved.
Comment thread .specs/badge.md Outdated
Comment thread packages/webkit/src/components/content/badge/badge.vue Outdated
Comment thread packages/webkit/src/components/content/badge/badge.vue Outdated
… includes

addresses review feedback — replaces the explicit severity if-chain with a validSeverities array + includes guard.
@robsongajunior robsongajunior merged commit 6bf958b into dev Jun 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants