Posted by Gábor Hojtsy on November 18, 2010 at 2:10pm
4 followers
| Project: | Wysiwyg |
| Version: | 7.x-2.x-dev |
| Component: | Editor - markItUp |
| Category: | bug report |
| Priority: | major |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Not sure what the 'theme' type in this piece of code supposed to convey, but there is no such type documented for Drupal 7, so it will just fall back to the default:
<?php
drupal_add_css($editor['library path'] . '/markitup/skins/' . $theme . '/style.css', 'theme');
?>The theme type meant a distinct thing in Drupal 6, not anymore in Drupal 7.
Comments
#1
Looks like something we overlooked when porting to D7, don't remember if that had already changed back then. We've recently gotten a few other issues related to D7 API changes but I've been too busy to look at them yet. :(
#2
D7 API changed since Wysiwyg was ported to it. We'll need to pass an array with a 'group' key with the value CSS_THEME instead. While we're tinkering with this, we might as well go all the way and use the 'basename' key to keep MarkItUp's style.css files from overriding Seven's style.css. That override makes the Seven theme look horrible so I'll bump this to major.
This patch should do the trick.
#3
Thanks for reporting, reviewing, and testing! Committed to attached patch to HEAD.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
#4
Automatically closed -- issue fixed for 2 weeks with no activity.