Skip to content
This repository has been archived by the owner on Nov 2, 2019. It is now read-only.

TinyMCE Usage Varies Between EDIT and ADD Actions #119

Open
maxdumas opened this issue Jul 2, 2015 · 8 comments
Open

TinyMCE Usage Varies Between EDIT and ADD Actions #119

maxdumas opened this issue Jul 2, 2015 · 8 comments

Comments

@maxdumas
Copy link
Member

maxdumas commented Jul 2, 2015

To reproduce, see edit for related versus add.

The "Name" field on the EDIT page uses a TinyMCE input (which is incorrect), while the ADD page uses a textarea, which is correct. This must have something to do with some incorrect binding of data when existing values are being loaded for the edit page, or some other incongruity between the two.

@ethanresnick
Copy link
Member

On my machine at least, the behavior on the edit and add pages is the same: both show the WYSIWG when they're not supposed to, but only about half the time. Try refreshing each page a bunch of times and I think you'll see that you can sometimes get the WYSIWG to show up (or not) on each. The data binding issue appears to be occurring now only when the WYSIWG is showing incorrectly, but I'm not 100% sure about that.

@maxdumas
Copy link
Member Author

maxdumas commented Jul 2, 2015

What the shit. My only explanation for this is that the server running the API or serving the intranet has developed free will and so is no longer deterministic.

@maxdumas maxdumas added medium and removed small labels Jul 2, 2015
@ethanresnick
Copy link
Member

Yeah. It's fucking weird. You can reproduce it, though, right?

@maxdumas
Copy link
Member Author

maxdumas commented Jul 2, 2015

Yeah I can. You're right, it's about 50% of the time. I'm going to try to look into this tomorrow.

@ethanresnick
Copy link
Member

👍 Thanks Max!

@freialobo
Copy link
Member

"My only explanation for this is that the server running the API or serving the intranet has developed free will and so is no longer deterministic." is the best thing ever omg.

@jhishan
Copy link

jhishan commented Jul 10, 2015

^LOL, we have some AI stuff going on here

@maxdumas
Copy link
Member Author

maxdumas commented Oct 2, 2015

Before I forget since I haven't had time to tackle this: The issue with this is that for some reason tinyMCE is converting all textarea fields on the page into tinyMCE inputs automatically, which I think screws up our model binding majorly and the API doesn't like it either. I couldn't figure out why this is happening — it doesn't look like we're setting anything on tinyMCE that would cause this. But this is definitely what is wrong.

I should also mention that I think this is why #125 is occurring.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants