bg_color and txt_color not working in text mode

[gesquive]

bg_color and txt_color not working in text mode

[cderv]

It seems to be different behavior on different navigator (IE, firefox). Something to do with the slate.js script which use those attributes. I may miss something

[cderv]

On Firefox, here is what I have when I am not providing an img to google music

-
  name: "google music"
  url: "https://play.google.com/music/listen"
  bg_color: "#ffffff"
  txt_color: "#ff5722"
  tags: ["music"]

Is there a problem with my version of Firefox or with the slate.js file ?

[gesquive]

From the look of that screenshot, it looks like the first div is being appended successfully and the second/third divs are not. You can see this part of the code uses the append function.

@cderv can you verify the version of firefox you are using? Also, are there any error messages printed to your console when you refresh?

[cderv]

I use Firefox 45.8.0. Could it come from this version ?
I find that you use a URL function that is experimental

Developper tool on firefox (F12) says that

• JS
  • HTML Encoding is not declared and should be
• CSS
  • some properties are unknown
    • -moz-osx-font-smoothing
    • user-select
    • background-position-x
    • background-position-y
    • background-repeat-x
    • background-repeat-y
• Journal
  • it display an error err:TypeError: s.append is not a function

I think this error is related to the append you mentionned. In the docs folder js/slate.js contain these lines with an s.append instead of domainDiv.append (see demo code for slate

What do you think is the problem ?

[gesquive]

Yup, its the append function. append only works on firefox v49+. I have modified the code to not use append and make it more backwards compatible. A function that requires browsers from late 2016 might be a little too new.

As for the URL function, even though it is experimental. It has been included in browsers since 2013, making it stable enough for me to use.

Thanks for the finding the bug.