TinyMCE is not generating a warning when loading in Safari - even thought the safari warning option has been set.

CommentFileSizeAuthor
#7 wysiwyg.safari.patch2.94 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Status: Active » Closed (works as designed)

As a Safari user, I implore you to NOT enable the warning. If you do, your site will be unusable for Safari users, as they will see a javascript dialog box on EVERY page. IMO, this Safari warning is a huge bug in TinyMCE (the editor code, not the Drupal module).

If the TinyMCE module is forcibly turning off the Safari warning, it's doing you a huge favor.

The right solution may be to disable the TinyMCE editor for Safari users. Which I believe is an option in the TinyMCE editor configuration (again, the editor code, not the Drupal module.)

AdrianB’s picture

AdrianB CreditAttribution: AdrianB commented

I think it would be best if there was an option in the module to disable TinyMCE for Safari users and then use the filtered html input filter istead of the rich text filter. But I don't know if that's possible.

jpsalter’s picture

jpsalter CreditAttribution: jpsalter commented
Status: Closed (works as designed) » Active

JohnAlbin and AdrianB - thank you for your input and clarifications.

1) Yes, it would be preferable to have TinyMCE turn off in Safari, but the option is not available in tinymce.module
2) Without this option - TinyMCE loads in Safari but does not work correctly - without the warning Safari users struggle and assume they are doing something wrong
3) Safari users need to be warned of this situation

Restating the bug: when the Safari warning option is set in the tinymce.module the Safari warning does not appear in TinyMCE when loaded in Safari.

Changing from "by design" -> "active"

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Title: Safari warning does not appear » Safari warning no longer supported
Status: Active » Postponed (maintainer needs more info)

Good news! The latest 2.1.0 version of the TinyMCE editor no longer includes the god-awful safari_warning alert box. (So, obviously, the safari warning option inside the TinyMCE module needs to be removed.)

The new version may be more compatible with Safari (see http://wiki.moxiecode.com/index.php/TinyMCE:Compatibility ). That page says it “works "okay".”

I haven‘t the time today to test the new 2.1.0 version with Safari. Can someone report back?

The only other option is to disable TinyMCE entirely for Safari using the browsers attribute inside tinyMCE.init().

jpsalter’s picture

jpsalter CreditAttribution: jpsalter commented

This is good news. I'll install, test, and report back.

The instructions to disable TinyMCE for Safari is here:
http://wiki.moxiecode.com/index.php/TinyMCE:Configuration/browsers

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Project: TinyMCE » Wysiwyg
Version: 4.7.x-1.x-dev » 6.x-1.x-dev
Category: bug » task
Status: Postponed (maintainer needs more info) » Active
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Active » Fixed
FileSize
wysiwyg.safari.patch2.94 KB

Committed attached patch, which removes the editor configuration option and value.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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