When installed, Whizzywig does not show anywhere an body textarea is invisible. When I click "disable richtext" body textarea appears with green font.

Tried with Whizzywig v63 and Whizzywig v60.
It also appears that Wysiwyg module tries to include xhtml.js file from "sites/all/libraries/whizzywig" but Whizzywig site says that this file is not needed anymore. When I include xhtml.js nothing changes.

CommentFileSizeAuthor
#2 wysiwyg-whizzywig-1207170.2.patch706 bytestwod

Comments

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Title: Whizzywig does not show » Whizzywig no longer needs xhtml.js

Hmm, that textarea styling should have been fixed with commit e1aae9... from #1079694: Disable rich text, textarea display is still set to none. If you're still seeing this, can you reopen that issue please?

It still including xhtml.js does need fixing, will do that when I get home from work tomorrow.

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
StatusFileSize
new wysiwyg-whizzywig-1207170.2.patch706 bytes

Hmm, this makes me wish our version overrides "cascaded" so we didn't have to redefine the 'libraries' key on each API change from now on...

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Version: 6.x-2.4 » 7.x-2.x-dev
Priority: Major » Normal
Status: Active » Needs review

Patch was made against 7.x-2.x but should apply cleanly to all versions.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Needs work

We need to do it reversed. At some point, we'll remove the old version definitions, only keeping the current. Hence, the default should be the new, previous versions get the override.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue summary: View changes
Parent issue: » #2343445: Stable release of Wysiwyg module, v7.x-2.4 please

  • TwoD committed 6b7fafa on 6.x-2.x 
    - #1207170 by TwoD: Whizzywig no longer needs xhtml.js.
  • TwoD committed 45af74b on 7.x-2.x
    - #1207170 by TwoD: Whizzywig no longer needs xhtml.js.
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Needs work » Fixed

Did what @sun said...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.