fix(jans-casa): primary button not changing color when customized via custom branding plugin

[mjatin-dev]

closes #10053

[dryrunsecurity]

DryRun Security Summary

The pull request modifies the CSS styles used in the "jans-casa" application, primarily focused on updating the visual styles of UI elements, without introducing any obvious security vulnerabilities, but the overall application's security posture should still be reviewed and tested to identify and address any potential security risks.

Expand for full summary

Summary:

The code changes in this pull request appear to be focused on modifying the CSS styles used in the "jans-casa" application. The changes are made in the "css-component-rules.properties" file, which likely contains reusable CSS rules for various UI components.

From a security perspective, the changes do not seem to introduce any obvious security vulnerabilities, as they are primarily focused on updating the visual styles of UI elements. However, it's important to note that while the code changes themselves may not directly impact security, the overall application's security posture should be considered. Factors such as input validation, access control, and secure coding practices should be reviewed to ensure the application is not vulnerable to common web application security issues (e.g., XSS, CSRF, SQL injection). Additionally, the use of third-party libraries or frameworks, if any, should be reviewed to ensure they are up-to-date and free from known vulnerabilities. Proper security testing, including both static and dynamic analysis, should be conducted to identify and address any potential security risks.

Files Changed:

• jans-casa/app/src/main/resources/css-component-rules.properties: This file contains the CSS styles used in the "jans-casa" application. The changes in this pull request appear to be focused on modifying the visual styles of various UI elements, such as buttons, inputs, and other components.

Code Analysis

We ran 9 analyzers against 1 file and 0 analyzers had findings. 9 analyzers had no findings.

Riskiness

🟢 Risk threshold not exceeded.

View PR in the DryRun Dashboard.

[jgomer2001]

Please remove z-button from tertiaryButton

Also, if btn was removed from primary and third, is there some reason for secondaryButton to have it ?

[sonarqubecloud]

Quality Gate passed for 'jans-cli'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[mjatin-dev]

z-button
I removed the z-button and also remove btn from secondary-button.