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

Preserve font properties when applied to all the document #17211

Closed
rvasseur31 opened this issue Oct 3, 2024 · 2 comments
Closed

Preserve font properties when applied to all the document #17211

rvasseur31 opened this issue Oct 3, 2024 · 2 comments
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:font resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@rvasseur31
Copy link

📝 Provide detailed reproduction steps (if any)

  1. Add text with one break line between each paragraph
  2. Select all document and apply a font property like font size, font family or bold
  3. Place the cursor on an empty line
  4. Write text

✔️ Expected result

The text should have the font properties that we applied to the all document

❌ Actual result

The text have the defaults properties. It might be liked to #17152

Screen.Recording.2024-10-03.at.10.16.24.mov

📃 Other details

  • Browser: Brave or Chrome
  • OS: MacOS Sonoma (Not tested on windows)
  • First affected CKEditor version: 43.1.0 (Not tested on previous versions)
  • Installed CKEditor plugins: Alignment, AutoLink, Autoformat, Bold, Essentials, FontBackgroundColor, FontColor, FontFamily, FontSize, GeneralHtmlSupport, Heading, Italic, Link, List, Paragraph, RemoveFormat, SelectAll, Strikethrough, Underline, Undo

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@rvasseur31 rvasseur31 added the type:bug This issue reports a buggy (incorrect) behavior. label Oct 3, 2024
@Witoso
Copy link
Member

Witoso commented Oct 14, 2024

I think there's a dup somewhere, but I couldn't find it. For the editor, the default is the stylesheets. Removal of the whole text is back to the defaults. We don't have a notion of changing dynamically the default font while interacting with fonts plugins (as it works in GDoc for example).

@Witoso Witoso added package:font domain:ui/ux This issue reports a problem related to UI or UX. labels Oct 14, 2024
@Witoso
Copy link
Member

Witoso commented Oct 14, 2024

Found it! DUP: #10517

@Witoso Witoso added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label Oct 14, 2024
@Witoso Witoso closed this as not planned Won't fix, can't repro, duplicate, stale Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:font resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

2 participants