Skip to content

refactor(list): rename item group header and list mixin - #21

Merged
rdlabo merged 3 commits into
mainfrom
docs/item-group-header-migration
Aug 21, 2026
Merged

refactor(list): rename item group header and list mixin#21
rdlabo merged 3 commits into
mainfrom
docs/item-group-header-migration

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • rename .header-item-group to .item-group-header across the demo and theme selector
  • rename utils/theme-list-inset.scss to utils/structured-list.scss
  • expose the internal mixin as structured-list.layout($outer-note-inset)
  • add an English migration guide with a copyable template diff
  • keep the repository guide out of the portal until it is registered there

Validation

  • npm run build
  • Prettier via lint-staged
  • verified that old class, module, and mixin names no longer remain in theme/demo sources
  • independent OSS documentation review approved

@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Playwright test results

passed  78 passed

Details

stats  78 tests across 1 suite
duration  1 minute, 28 seconds
commit  97204b5
info  This detailed result covers Ionic 9 only. Ionic 8 runs against the same screenshots in a separate matrix job; check the workflow run for both results. To update the screenshots, comment with /update-screenshots.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-md3/pr-21/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 21, 2026
@rdlabo rdlabo changed the title docs: document item group header class rename refactor(list): rename item group header and list mixin Aug 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-md3/pr-21/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-md3/pr-21/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 21, 2026
@rdlabo
rdlabo marked this pull request as ready for review August 21, 2026 23:58
@rdlabo
rdlabo merged commit d02990c into main Aug 21, 2026
10 checks passed
@rdlabo
rdlabo deleted the docs/item-group-header-migration branch August 21, 2026 23:58
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