Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance(alert): Add supporting custom styles to ngxsmart #88

Merged
merged 7 commits into from
Nov 11, 2023

Conversation

pavankjadda
Copy link
Contributor

No description provided.

This commit adds a new playwright.config.ts file for Playwright end-to-end testing configurations. Moreover, it updates various dependencies to the latest version in package.json and updates the lock file accordingly. Jasmine and Karma related dependencies are replaced with Jest and Playwright related ones.
Modified the main ESLint configuration, .eslintrc.json, to refine the ignore patterns and file overrides. A new rules configuration file for ngxsmart project has been added. Several new plugins have been installed, including eslint-config-prettier, eslint-plugin-import, eslint-plugin-jsdoc and eslint-plugin-prefer-arrow, to improve code quality. Changes have been reflected in both package.json and pnpm-lock.yaml.
Added `ts-ignore`, updated an event parameter, and enabled strict mode in the TSConfig. These essential modifications improve code robustness and ensure compliance to TypeScript's strict type-checking for overall better maintainability.
Added a new @input() property 'class' to alert.component in ngxsmart to allow addition of custom classes to the alert component. This would enable developers to apply more specific styles on the component if needed. The usage is updated in the demo HTML file. Also, minor adjustments are made in the alert-demo.component like changing the initially set dismissOnTimeout property to false, probably done for testing the new feature.
@pavankjadda pavankjadda added enhancement New feature or request alert Alert Component labels Nov 11, 2023
@pavankjadda pavankjadda self-assigned this Nov 11, 2023
Copy link

netlify bot commented Nov 11, 2023

Deploy Preview for js-smart-ngxsmart failed.

Name Link
🔨 Latest commit 80c92ad
🔍 Latest deploy log https://app.netlify.com/sites/js-smart-ngxsmart/deploys/654facef12f8fd000869bab2

@pavankjadda pavankjadda merged commit ca2ad5c into main Nov 11, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alert Alert Component enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant