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

Check it out button in editing a WYSIWYG field has broken styles #27341

Closed
zJaaal opened this issue Jan 16, 2024 · 2 comments · Fixed by #30250
Closed

Check it out button in editing a WYSIWYG field has broken styles #27341

zJaaal opened this issue Jan 16, 2024 · 2 comments · Fixed by #30250

Comments

@zJaaal
Copy link
Contributor

zJaaal commented Jan 16, 2024

Parent Issue

No response

Problem Statement

The styles for the Check it out button on the Block Editor information for WYSIWYG are off, The label has an incorrect color.

This just happens when editing a WYSIWYG field, when we create a new one the label has the correct color.

Steps to Reproduce

Screen.Recording.2024-01-16.at.4.45.16.PM.mov

Acceptance Criteria

As an user I want to see the label with the correct color and the correct contrast.

dotCMS Version

master-latest

Proposed Objective

User Experience

Proposed Priority

Priority 4 - Trivial

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@oidacra
Copy link
Member

oidacra commented Oct 4, 2024

close in favor of #30248

@oidacra oidacra closed this as completed Oct 4, 2024
@oidacra oidacra reopened this Oct 4, 2024
@nollymar nollymar closed this as completed Oct 4, 2024
@valentinogiardino valentinogiardino self-assigned this Oct 4, 2024
github-merge-queue bot pushed a commit that referenced this issue Oct 4, 2024
…#27341  (#30250)

### Proposed Changes
* Updated the `dot-convert-to-block-info.component.html` template to use
the `label` property of PrimeNG `pButton` instead of manually setting
the button label inside the button element.
* Imported the `ButtonModule` from PrimeNG in the
`dot-convert-to-block-info.component.spec.ts` to properly support the
`pButton` directive and its `label` property in tests.

### Checklist
- [x] Tests
- [x] Translations
- [x] Security Implications Contemplated.

### Additional Info
This update ensures that the `pButton` directive from PrimeNG correctly
handles the button label by using the `label` property, allowing for
more maintainable and PrimeNG-compliant code. Previously, the button
label was manually rendered inside the button, which lead to styling
issues.

### Screenshots
Original             |  Updated
:-------------------------:|:-------------------------:
<img width="1383" alt="image"
src="https://github.com/user-attachments/assets/6e9de521-6826-4a12-845d-08cc22a34764">
| <img width="1124" alt="image"
src="https://github.com/user-attachments/assets/134bab72-ba5c-49e9-a60f-d4a9b2f3f5f3">
@valentinogiardino valentinogiardino removed their assignment Oct 4, 2024
@jgambarios jgambarios self-assigned this Oct 7, 2024
@bryanboza
Copy link
Member

Fixed, Unable to reproduce after this fix
Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

10 participants