Skip to content

fix(loading): isolate service overlay CSS from loading buttons - #53

Merged
sdcorejs merged 3 commits into
mainfrom
fix/loading-overlay-2.8
Sep 10, 2026
Merged

fix(loading): isolate service overlay CSS from loading buttons#53
sdcorejs merged 3 commits into
mainfrom
fix/loading-overlay-2.8

Conversation

@sdcorejs

Copy link
Copy Markdown
Owner

Problem and change

SdLoadingService global .sd-loading rules also selected loading buttons, stretching and repositioning header actions. Mark service-owned overlays with data-sd-loading-overlay and scope overlay/spinner rules to those elements. Preserve button behavior and loading ownership/ref counting.

Validation

  • Chrome rendering regression reproduces the old failure; full canonical suite after main integration: 5,336 pass.
  • Earlier loading preflight: all four full suites (21,332 tests), release lint, Showcase tests/build passed.
  • Integrated main: four builds/packs, package contracts and strict consumer compiles, 158 script tests and sync guard passed.
  • Console patch removes only loading; retained layout/tab-router hunks apply to rebuilt 20.2.8.

Release

Prepares suffix 2.8 across Angular 19–22. Includes existing main table/data-state/upload changes; the upload-file API removal remains documented as breaking. No npm publication or release tag. Consumer upgrade instructions and retained patch are in .sdcorejs/docs/angular/loading-overlay-2.8/.

Keep button state/classes and loading reference ownership intact. Prepare the 2.8 release transaction and Console patch upgrade handoff.

Verification: 5333 Chrome tests on each Angular line; 206 Showcase tests; full lint; sync; 158 script tests; four build/pack artifacts and strict consumers.
# Conflicts:
#	CHANGELOG.md
#	versions/v19/SYNC-STATUS.md
#	versions/v20/SYNC-STATUS.md
#	versions/v21/SYNC-STATUS.md
#	versions/v22/SYNC-STATUS.md
@sdcorejs
sdcorejs merged commit f65ba0f into main Sep 10, 2026
6 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