Skip to content

FIX: Ckeditor padding + heading sizes#7368

Draft
Timo-Breumelhof wants to merge 2 commits into
dnnsoftware:developfrom
Timo-Breumelhof:feature/7367-Editor-config
Draft

FIX: Ckeditor padding + heading sizes#7368
Timo-Breumelhof wants to merge 2 commits into
dnnsoftware:developfrom
Timo-Breumelhof:feature/7367-Editor-config

Conversation

@Timo-Breumelhof

@Timo-Breumelhof Timo-Breumelhof commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

2 Commits.
A. Very simple add some padding to the CkEditor screen
B. For the heading sizes, ended up being a bit more complex.

Summary

About the Heading size issue.

I noticed this:

image

A difference in heading sizes.
dnnFormSectionHead can contain a link or a span

The second one contains a span the others a link.

Span size:

Anchor size:

This is why they are different.
If you look at the interface, the heading a really large and IMO the size of the span is actually better than the ones containing a link. In edit users you get this:

Which looks weird IMO

image

So I moved what was in forms.scss into default.scss (not the best place but at least in one place)
And made them both the same size in rem.

There's a lot more one could changes here, but for now this at least fixes the issue of the header sizes.

FYI,

Manage users after this change:

image

Ckeditor manager

image

fixes #7367

@Timo-Breumelhof Timo-Breumelhof changed the title Feature/7367 editor config FIX: Ckeditor padding + heading sizes Jul 6, 2026
@valadas valadas added this to the 10.3.3 milestone Jul 10, 2026

@valadas valadas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This technically affects more than the problem in question and technically is a bit of a change for all usages of the ui/ux pattern. However I do believe it is the right thing to do as those headers were incorrectly huge everywhere. So my vote goes to just go ahead with this change.

@valadas

valadas commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

@Timo-Breumelhof this is marked as draft, is that on purpose because you are still working on it w you want to mark this ready for review?

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

@valadas yes, I still need to check something but ran out of time, I'll finish this one next week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement]: CkEditor Config screen Layout

2 participants