Download & Extend

WYSIWYG ignores RTL CSS when using Theme CSS as Editor CSS

Project:Wysiwyg
Version:7.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

wysiwyg_get_css() ignores RTL css when invoking drupal_add_css().

This can be reproduced if you have chose the "Use theme CSS" as the "Editor CSS". And you have a -rtl.css file in your theme, this is when the editor ignores the -rtl.css files.

Patch is attached in comment #1

Comments

#1

Status:active» needs review
AttachmentSize
WYSIWYG_ignores_RTL_CSS-1926584-1.patch 493 bytes

#2

Hmm, Locale module switches to the RTL stylesheets based on the current Drupal language, but we don't currently use that elsewhere, so we'd also need to fix #532794: Use current user's language together with this for the rest of the editor to actually reflect being used in RTL mode.

#3

Well this patch only adds drupal_alter('css') which happend to be used by Locale module to inject -rtl.css styles.
This means that this will allow any other modules implementing hook_css_alter() to reflect their changes to WYSIWYG if it was theme related CSS.

Maybe the title of the issue is a bit specific for Locale's case, but this indeed will help other modules achieve their purpose in WYSIWYG.

nobody click here