Conversation
… improved UX - Replaced div elements with p-card for render mode options to enhance visual consistency. - Adjusted CSS classes for better alignment with Tailwind CSS standards. - Updated error and success message styling in the login component for clearer user feedback. - Removed unused background color property in navigation header styles. These changes aim to streamline the user interface and improve overall user experience across components.
github-actions
bot
added
the
Area : Frontend
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors a few UI components in dotcms-ui to improve visual consistency and UX by aligning render mode selection with PrimeNG card styling, updating message color feedback on the login page, and simplifying navigation header styling.
Changes:
- Replaced render mode option containers with
p-cardand updated Tailwind utility classes. - Updated login message styling to use Tailwind text color utilities instead of
ngClass. - Removed an unused
background-colorfrom the nav header button wrapper styles.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| core-web/apps/dotcms-ui/src/app/view/components/login/dot-login-component/dot-login.component.html | Adjusts error/success message styling via Tailwind class bindings. |
| core-web/apps/dotcms-ui/src/app/view/components/dot-navigation/components/dot-nav-header/dot-nav-header.component.scss | Removes a background color from the nav header button wrapper. |
| core-web/apps/dotcms-ui/src/app/portlets/shared/dot-content-types-edit/components/fields/content-type-fields-properties-form/field-properties/render-mode-property/render-mode-property.component.html | Refactors render mode selection UI to use p-card with updated Tailwind classes. |
...ds-properties-form/field-properties/render-mode-property/render-mode-property.component.html
Show resolved
Hide resolved
...eb/apps/dotcms-ui/src/app/view/components/login/dot-login-component/dot-login.component.html
Show resolved
Hide resolved
adrianjm-dotCMS
approved these changes
Mar 3, 2026
hmoreras
approved these changes
Mar 3, 2026
oidacra
approved these changes
Mar 3, 2026
… Tailwind CSS classes
nicobytes
deleted the
34825-ui-inconsistencies-in-login-toggle-cards-and-sidebar
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… improved UX
These changes aim to streamline the user interface and improve overall user experience across components.
Screenshots
Related #34825
This PR fixes: #34825