Skip to content

Conversation

valorkin
Copy link
Member

Summary

  • Replace deprecated @import with modern @use statements in all SCSS files
  • Fix Sass deprecation warnings during build process
  • Use as * syntax for variables to maintain existing functionality
  • Move @use statements to top of files as required by Sass specification

Changes Made

  • Updated style.scss, sidebar.scss, common-styles.scss, and adaptive.scss
  • Updated datepicker SCSS files: bs-datepicker.scss and mixins.scss
  • Eliminated all Sass @import deprecation warnings from build output

Test plan

  • Build completes successfully without deprecation warnings
  • All lint checks pass
  • All unit tests pass
  • Styles render correctly (no visual regressions)

🤖 Generated with Claude Code

- Replace @import with @use statements in all SCSS files to eliminate deprecation warnings
- Use 'as *' syntax for variables to maintain existing functionality
- Move @use statements to top of files as required by Sass specification
- Fixes Sass deprecation warnings during build process

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link

cloudflare-workers-and-pages bot commented May 26, 2025

Deploying ngx-bootstrap with  Cloudflare Pages  Cloudflare Pages

Latest commit: f116362
Status: ✅  Deploy successful!
Preview URL: https://524db15d.ngx-bootstrap-avf.pages.dev
Branch Preview URL: https://fix-build-warnings.ngx-bootstrap-avf.pages.dev

View logs

valorkin and others added 8 commits May 27, 2025 01:50
- Added platform browser checks in AppComponent to prevent SSR access to window globals
- Added platform browser checks in AddNavComponent for DOM manipulation and scroll events
- Added platform browser checks in SidebarComponent for window.innerWidth access
- Fixed theme-provider _guessBsVersion function to handle SSR gracefully by checking for document availability
- Ensured all browser-specific APIs are only accessed when running in browser context

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
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