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

fix the issue #973 #983

Merged
merged 1 commit into from
Oct 17, 2024
Merged

fix the issue #973 #983

merged 1 commit into from
Oct 17, 2024

Conversation

Jameslebron6
Copy link
Contributor

@Jameslebron6 Jameslebron6 commented Oct 16, 2024

Issue

WYSIWYG: When text is set to bold, it is not possible to change its color. #973

Pull Request Description

Summary
This pull request addresses the issue #973 where, after applying bold formatting to text in the WYSIWYG editor, users were unable to change the color of the bolded text.

Changes Made

Updated the text formatting logic to allow color changes to be applied to bolded text.
Ensured that the editor state correctly reflects both the bold and color attributes simultaneously.

Steps to Test

1.Open the WYSIWYG text editor.
2.Select some text and click the Bold button to apply bold formatting.
3.Then select the same text, click the Text Color button, and choose a color.
4.Verify that the text remains bold while the selected color is applied.
5.This fix ensures that users can now change the color of bold text without losing the bold formatting.

Outcome Screenshots

Screen.Recording.2024-10-16.at.9.50.20.pm.mov

Additional note

1.This modification allows the text font to change its color while being bold, fixing the previous bug

2.Please review the changes and let me know if any further modifications are needed.

Thank you for considering this update!

@Jameslebron6
Copy link
Contributor Author

Jameslebron6 commented Oct 16, 2024

Dear @zoltanszogyenyi
Hi, I just improved the wysiwyg plugin page in the issue of #973: When text is set to bold, it is no longer possible to set a color for it. in PR #983. It would be thankful is you can review my PR and merge it.

@Jameslebron6
Copy link
Contributor Author

Jameslebron6 commented Oct 16, 2024

Original, it will show like the below Screenshots

Screen.Recording.2024-10-16.at.10.29.31.pm.mov

@zoltanszogyenyi
Copy link
Member

This is a good fix for an edge case. Thanks for contributing! <3

@zoltanszogyenyi zoltanszogyenyi merged commit 05cc899 into themesberg:main Oct 17, 2024
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.

2 participants