Skip to content

Restore parent module guards for child components - #355

Merged
damyanpetev merged 3 commits into
dpetev/child-modules-cleanupfrom
copilot/update-refactor-ensuremodulesloaded
Aug 21, 2026
Merged

Restore parent module guards for child components#355
damyanpetev merged 3 commits into
dpetev/child-modules-cleanupfrom
copilot/update-refactor-ensuremodulesloaded

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Dedicated child components now retain lazy module registration while targeting their parent component modules instead of redundant child modules.

  • Parent module registration
    • Replaced child Igb*Module guards with the corresponding parent module across card, carousel, dropdown, list, nav drawer, rating, select, slider, stepper, tile manager, button group, and tree children.
    • Preserves the existing IsLoadRequested/Register lazy-loading pattern.
if (!IgbCardModule.IsLoadRequested(IgBlazor))
{
    IgbCardModule.Register(IgBlazor);
}

Co-authored-by: damyanpetev <3198469+damyanpetev@users.noreply.github.com>
Copilot AI changed the title [WIP] Update EnsureModulesLoaded method for child components Restore parent module guards for child components Aug 20, 2026
Copilot AI requested a review from damyanpetev August 20, 2026 16:46
@damyanpetev
damyanpetev changed the base branch from master to dpetev/child-modules-cleanup August 21, 2026 05:57
…to copilot/update-refactor-ensuremodulesloaded

# Conflicts:
#	src/components/Blazor/CardActions.cs
#	src/components/Blazor/CardContent.cs
#	src/components/Blazor/CardHeader.cs
#	src/components/Blazor/CardMedia.cs
#	src/components/Blazor/CarouselIndicator.cs
#	src/components/Blazor/CarouselSlide.cs
#	src/components/Blazor/DropdownGroup.cs
#	src/components/Blazor/DropdownHeader.cs
#	src/components/Blazor/DropdownItem.cs
#	src/components/Blazor/ListHeader.cs
#	src/components/Blazor/ListItem.cs
#	src/components/Blazor/NavDrawerHeaderItem.cs
#	src/components/Blazor/NavDrawerItem.cs
#	src/components/Blazor/RatingSymbol.cs
#	src/components/Blazor/SelectGroup.cs
#	src/components/Blazor/SelectHeader.cs
#	src/components/Blazor/SelectItem.cs
#	src/components/Blazor/SliderLabel.cs
#	src/components/Blazor/Step.cs
#	src/components/Blazor/Tile.cs
#	src/components/Blazor/ToggleButton.cs
#	src/components/Blazor/TreeItem.cs

Co-authored-by: damyanpetev <3198469+damyanpetev@users.noreply.github.com>
@damyanpetev
damyanpetev marked this pull request as ready for review August 21, 2026 06:11
@damyanpetev
damyanpetev merged commit 951ad76 into dpetev/child-modules-cleanup Aug 21, 2026
@damyanpetev
damyanpetev deleted the copilot/update-refactor-ensuremodulesloaded branch August 21, 2026 06:12
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.

2 participants