Skip to content

Require grouped Ionic list items - #28

Merged
rdlabo merged 6 commits into
mainfrom
feat/require-ion-item-group
Aug 22, 2026
Merged

Require grouped Ionic list items#28
rdlabo merged 6 commits into
mainfrom
feat/require-ion-item-group

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add the recommended require-ion-item-group template rule for iOS 26 and Material Design 3 list structure
  • require ion-list items to use ion-item-group, ion-reorder-group, ion-radio-group, or ion-accordion within ion-accordion-group
  • treat Angular control-flow blocks, ng-container, and ng-template as transparent while rejecting intervening rendered elements
  • cover @empty and the @defer placeholder, loading, and error blocks
  • wrap safely groupable list contents in one ion-item-group with --fix when the same template already uses ion-item-group; otherwise offer a suggestion that reminds users to add the standalone import
  • skip fixes for mixed, nested, dynamic, reusable-template, or otherwise ambiguous rendered content
  • share Angular template AST traversal across the new rule and existing template rules, including all control-flow secondary blocks and duplicate-node protection
  • publish the rule through generated exports/config and document it in README and rule docs

Verification

  • npm run update (idempotent)
  • npm run lint
  • npm test (27 suites, 575 tests)

@rdlabo
rdlabo marked this pull request as ready for review August 22, 2026 03:56
@rdlabo
rdlabo merged commit bde8a48 into main Aug 22, 2026
5 checks passed
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