Opened 13 years ago
Closed 13 years ago
#23551 closed defect (bug) (fixed)
Twenty Thirteen: add Editor styles
| Reported by: | lancewillett | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Bundled Theme | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description (last modified by )
Add styles for the TinyMCE visual editor.
Note: see #22499 for loading the Google fonts in the editor stylesheet in Twenty Thirteen.
Attachments (5)
Change History (21)
#5
@
13 years ago
Noting also Editor Styles will need RTL version in css/editor-style-rtl.css, see also #23550.
#6
@
13 years ago
.4 patch: update comment blocks (add a few periods, remove extra space before closing braces) and update CSS comment style to match main stylesheet.
#8
in reply to: ↑ 7
@
13 years ago
Replying to azaozz:
Would there be a visible indication of the theme's width in the editor?
We didn't do it in older default themes -- should we add it? I personally like it without.
#9
@
13 years ago
It's always been a bit weird that there is a well defined box where to type but the text wraps in the middle of it. On the other hand most users are used to that by now.
#10
@
13 years ago
- Cc added
@kovshenin Do you have an opinion on the shaded background colors to show an idea of the content width?
#11
@
13 years ago
I don't have a strong opinion on this. I can usually figure out the content width right after typing in the first line, and in DFW the dotted border is very helpful too. I also don't think it will hurt, even if people are used to it.
#13
@
13 years ago
@azaozz: On the other hand most users are used to that by now
That wrap still feels a little weird. I think I like the very light grey background.
Though if it's loaded by the theme, it will feel strange to them if/when they switch themes and it goes away.
#14
@
13 years ago
As discussed in today's Twenty Thirteen office hours log -- another suggestion is to use a dotted gray line on the right side, similar style to the DFW interface.
Sounds like most people voted to leave it out entirely, though.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

Used the Layout Test from the Theme Unit Test to verify the editor styles. There a still some issues with the
kdbandtttags though.