👍 force_hex_color - Converts background-color:transparent to background-color:#000000 (black) about tinymce HOT 11 OPEN

@TheSpyder

It happens in both 6 and 7.

Here's a fiddle:
https://fiddle.tiny.cloud/fVQGGgyZ0B/4

It shows both issues, the conversion to black and ignoring the style tags.

Source View

WYSIYG View

** Edit: I updated the fiddle and images to be more clear to explain what I mean

from tinymce.

I doing this in the meantime, it only addresses the issue of transparent being converted to black. The issue where it ignores and doesn't force HEX colors in the style tags, this won't address.

 editor.on('BeforeSetContent', function (e) {
         e.content = e.content.replace(/((color|background-color|border-color)\s*:\s*|background\s*:[^;]*?)\btransparent\b/g, '$1#00000000');
 });

from tinymce.

I can confirm the issue after upgrading to v7.0.0.
Maybe because the option was removed entirely and forced now?
Is there a way of not enforcing it via the init config?

EDIT: this solved my issue as well:

editor.on('BeforeSetContent', function (e) {
  e.content = e.content.replace(
    /((color|background-color|border-color)\s*:\s*|background\s*:[^;]*?)\btransparent\b/g,
    '$1#00000000'
  );
});

from tinymce.

Which version? A replication case will help a lot. Although the option is named "force hex color", the fact that it forces the color value to hex seems to be by design.

The option was added to solve a regression between TinyMCE 5 and 6, but we couldn't revert it as that would be a breaking change. We're looking at a proper fix for TinyMCE 7, so your feedback and an example would be much appreciated.

from tinymce.

I can confirm the issue existing in v7.0.0. We are using @tinymce/tinymce-react.

@TopCoder02's approach has helped.

from tinymce.

Confirmed this as a bug and a regression so escalating internally.

from tinymce.

Also encoutered it.

To summarize for people passing by (correct me if I’m wrong)

• The bug already exists in 6.x (if you the force_hex_color option)
• Moste people will encouter this bug when switching to 7.x (since the force_hex_color behavior is now enforced)

Anyway, thanks guys for your reactivity,
Tiny rocks ! 😎

from tinymce.

yes we haven't implemented the change yet so 7-dev will be the same as 6.

from tinymce.

Confirmed fixed in the next release. #9414

This can be confirmed using the 7-dev channel on fiddle.tiny.cloud

from tinymce.

@TheSpyder This changed doesn't address any of the issues reported here.

from tinymce.

Huh. I thought it was rolling back to the v5 implementation, but I see what you mean. Apologies.

from tinymce.