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

DONT MERGE: Primeng 18 with Lara Theme #31321

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

DONT MERGE: Primeng 18 with Lara Theme #31321

wants to merge 13 commits into from

Conversation

fmontes
Copy link
Member

@fmontes fmontes commented Feb 6, 2025

PrimeNG Components Modernization and UI Improvements

Summary

This PR includes a comprehensive update to modernize our PrimeNG implementation, improve component accessibility, and clean up legacy code. The changes focus on updating deprecated modules, enhancing UI consistency, and improving the overall developer experience.

Changes

  • Updated TabView implementation to use modern p-tabs classes across components
  • Replaced deprecated PrimeNG modules with their modern counterparts:
    • InputTextareaModuleTextareaModule
    • DynamicDialogModuleDynamicDialog
    • ChipsModuleChip in autocomplete tags
  • Enhanced form controls accessibility and layout:
    • Improved checkbox and radio button implementations
    • Removed redundant label prop from p-checkbox components
  • Updated DotAlertConfirmComponent method implementations
  • Implemented Lato font and refreshed UI styling
  • Cleaned up cursor-related code for better maintainability

Additional Notes

These changes are part of our ongoing effort to modernize our UI components and maintain consistency with PrimeNG's latest best practices. The updates should improve both the developer experience and end-user accessibility.

@fmontes fmontes requested review from a team, oidacra, nicobytes, valentinogiardino, zJaaal, rjvelazco, KevinDavilaDotCMS and hmoreras and removed request for a team February 6, 2025 21:57
@semgrep-code-dotcms-test
Copy link

Semgrep found 1 ssc-cee3e6d5-d7c8-4c35-9815-076aa1ebfd49 finding:

Risk: Affected versions of rollup are vulnerable to Improper Neutralization of Input During Web Page Generation ('Cross-site Scripting').

Manual Review Advice: A vulnerability from this advisory is reachable if you use Rollup to bundle JavaScript with import.meta.url and the output format is set to cjs, umd, or iife formats, while allowing users to inject scriptless HTML elements with unsanitized name attributes

Fix: Upgrade this library to at least version 4.22.4 at core/core-web/yarn.lock:20607.

Reference(s): GHSA-gcx4-mw62-g8wm, CVE-2024-47068

@@ -19,7 +19,7 @@ export default [
'/api/vtl',
'/tinymce'
],
target: 'http://localhost:8080',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beware of this

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is the URL of the proxy.

@fmontes fmontes changed the title Primeng 18 with Lara Theme DONT MERGE: Primeng 18 with Lara Theme Feb 11, 2025
Copy link

Please use a Conventional Commit title format for this PR. For more information, see https://www.conventionalcommits.org/en/v1.0.0/

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.

3 participants