feat(*): update Angular, Drop dialog support, move from scss to css

BREAKING CHANGE: Clients need to update to Angular 20 in order to use this version. As I wrote last october in #3, Dialog and Popup support were finally dropped. Only the "inline" mode is supported from now on. Additionally, probably irrelevant for clients, this library switched from scss to css which simplifies the build process.

Author: NetWin

.github/workflows/publish.yml

@@ -40,8 +40,7 @@ jobs:
 
       - name: Build library
         run: |
-          npm run build:lib:linux
-          npm run build:css:linux
+          npm run build
 
       - name: Release
         env:

.lintstagedrc

@@ -7,7 +7,7 @@
     "eslint --fix --cache",
     "prettier --write --cache"
   ],
-  "*.scss": [
+  "*.css": [
     "prettier --write --cache"
   ]
 }

LICENSE

@@ -2,7 +2,7 @@ The MIT License
 
 Copyright (c) 2014-2016 Daniel YK Pan.
 Copyright (c) 2020 Daniel Moncada.
-Copyright (c) 2024 NetWin.
+Copyright (c) 2024-2025 NetWin.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

angular.json

@@ -1,36 +1,56 @@
 {
   "$schema": "./node_modules/@angular/cli/lib/config/schema.json",
   "version": 1,
-  "cli": {
-    "cache": {
-      "enabled": true,
-      "path": ".cache",
-      "environment": "all"
+  "newProjectRoot": "projects",
+  "schematics": {
+    "@schematics/angular:component": {
+      "type": "component",
+      "style": "css"
+    },
+    "@schematics/angular:directive": {
+      "type": "directive"
+    },
+    "@schematics/angular:service": {
+      "type": "service"
+    },
+    "@schematics/angular:guard": {
+      "typeSeparator": "."
+    },
+    "@schematics/angular:interceptor": {
+      "typeSeparator": "."
+    },
+    "@schematics/angular:module": {
+      "typeSeparator": "."
+    },
+    "@schematics/angular:pipe": {
+      "typeSeparator": "."
+    },
+    "@schematics/angular:resolver": {
+      "typeSeparator": "."
     }
   },
-  "newProjectRoot": "projects",
   "projects": {
     "docs": {
       "root": "",
       "sourceRoot": "src",
       "projectType": "application",
       "prefix": "owl",
-      "schematics": {
-        "@schematics/angular:component": {
-          "style": "scss"
-        }
-      },
+      "schematics": {},
       "architect": {
         "build": {
           "builder": "@ng-doc/builder:application",
           "options": {
-            "allowedCommonJsDependencies": ["@ng-doc/core"],
+            "allowedCommonJsDependencies": [
+              "@ng-doc/core"
+            ],
             "aot": true,
             "outputPath": {
               "base": "dist/docs"
             },
             "index": "src/index.html",
-            "polyfills": ["src/polyfills.ts"],
+            "polyfills": [
+              "src/polyfills.ts"
+            ],
             "tsConfig": "src/tsconfig.app.json",
             "assets": [
               {
@@ -51,7 +71,10 @@
               "src/favicon.ico",
               "src/assets"
             ],
-            "styles": ["node_modules/@ng-doc/app/styles/global.css", "src/styles.scss"],
+            "styles": [
+              "node_modules/@ng-doc/app/styles/global.css",
+              "src/styles.css"
+            ],
             "scripts": [],
             "browser": "src/main.ts"
           },
@@ -68,6 +91,7 @@
               "sourceMap": false,
               "namedChunks": false,
               "aot": true,
+              "hmr": true,
               "extractLicenses": true,
               "budgets": [
                 {
@@ -114,15 +138,18 @@
           "defaultConfiguration": "development"
         },
         "extract-i18n": {
-          "builder": "@angular-devkit/build-angular:extract-i18n",
+          "builder": "@angular/build:extract-i18n",
           "options": {
             "buildTarget": "docs:build"
           }
         },
         "lint": {
           "builder": "@angular-eslint/builder:lint",
           "options": {
-            "lintFilePatterns": ["src/**/*.ts", "src/**/*.html"],
+            "lintFilePatterns": [
+              "src/**/*.ts",
+              "src/**/*.html"
+            ],
             "eslintConfig": "./eslint.config.js"
           }
         }
@@ -135,7 +162,7 @@
       "prefix": "owl",
       "architect": {
         "build": {
-          "builder": "@angular-devkit/build-angular:ng-packagr",
+          "builder": "@angular/build:ng-packagr",
           "options": {
             "project": "projects/picker/ng-package.json"
           },
@@ -150,16 +177,22 @@
           "defaultConfiguration": "production"
         },
         "test": {
-          "builder": "@angular-devkit/build-angular:karma",
+          "builder": "@angular/build:karma",
           "options": {
             "tsConfig": "projects/picker/tsconfig.spec.json",
-            "polyfills": ["zone.js", "zone.js/testing"]
+            "polyfills": [
+              "zone.js",
+              "zone.js/testing"
+            ]
           }
         },
         "lint": {
           "builder": "@angular-eslint/builder:lint",
           "options": {
-            "lintFilePatterns": ["projects/picker/src/lib/**/*.ts", "projects/picker/src/lib/**/*.html"],
+            "lintFilePatterns": [
+              "projects/picker/src/lib/**/*.ts",
+              "projects/picker/src/lib/**/*.html"
+            ],
             "eslintConfig": "./eslint.config.js"
           }
         }

eslint.config.js

@@ -6,6 +6,9 @@ const tsEslint = require('typescript-eslint');
 const stylistic = require('@stylistic/eslint-plugin');
 
 module.exports = tsEslint.config(
+  {
+    ignores: ['**/node_modules/', '**/dist/', '**/.angular/', '**/ng-doc/']
+  },
   {
     files: ['**/*.ts'],
     extends: [
@@ -69,11 +72,10 @@ module.exports = tsEslint.config(
       '@typescript-eslint/no-explicit-any': 'warn',
       // https://typescript-eslint.io/rules/prefer-optional-chain
       // Prefer `a?.b?.c` over `a && a.b && a.b.c`
-      '@typescript-eslint/prefer-optional-chain': 'warn',
+      '@typescript-eslint/prefer-optional-chain': 'error',
       // https://typescript-eslint.io/rules/consistent-type-definitions
       // Disables the rule that enforces consistent usage of either "type" or "interface" for type definitions
-      // TODO: enable again
-      '@typescript-eslint/consistent-type-definitions': 'off',
+      '@typescript-eslint/consistent-type-definitions': ['error', 'type'],
       // https://typescript-eslint.io/rules/explicit-function-return-type
       // Require explicit return types on functions and class methods
       '@typescript-eslint/explicit-function-return-type': ['warn', { 'allowExpressions': true }],
@@ -85,7 +87,7 @@ module.exports = tsEslint.config(
       ],
       // https://typescript-eslint.io/rules/no-empty-object-type
       // Disallow the `{}` type in object type annotations
-      '@typescript-eslint/no-empty-object-type': 'warn',
+      '@typescript-eslint/no-empty-object-type': 'error',
       // https://typescript-eslint.io/rules/consistent-indexed-object-style
       // Disables the rule that enforces consistent usage of either index signature or record types
       '@typescript-eslint/consistent-indexed-object-style': 'off',
@@ -97,7 +99,7 @@ module.exports = tsEslint.config(
       '@typescript-eslint/no-misused-promises': 'error',
       // https://typescript-eslint.io/rules/no-floating-promises
       // Disallow floating (unused / unawaited) promises
-      '@typescript-eslint/no-floating-promises': 'warn',
+      '@typescript-eslint/no-floating-promises': 'error',
       // https://typescript-eslint.io/rules/no-unused-vars
       // Disallow unused variables
       '@typescript-eslint/no-unused-vars': ['warn', { 'argsIgnorePattern': '^_', 'varsIgnorePattern': '^_' }],