search

globalvoices / NewsFrames-Code

Code for the NewsFrames Platform available at newsframes.org
https://newsframes.globalvoices.org
Other
2 stars 1 forks source link

MC Word image not inserted at cursor position and strange behaviour of jumping cursor #6

Open shrutimehta13 opened 6 years ago

shrutimehta13 commented 6 years ago

wrt Download Media Cloud Word Clouds as .pngs and/or .svgs - 10 HOURS #254

Browser affected: Chrome, Firefox

  1. create a pad and enter few paragraphs. This is Para1. This is Para2. This is Para3.
  2. Click Media Cloud from the right sidebar and perform a search.
  3. Place cursor say before word Para2 and choose 'Add to Pad' from MC frequency options.

Observed Result:

  1. Word Cloud image gets added to the beginning of the para instead of the cursor position.
  2. after that, if we try to place the cursor anywhere in 'this' paragraph, it keeps Jumping to preceding chars one by one. Cannot use keyboard arrow keys > to reach the desired cursor position, cursor jumps back to reach the first position. weird jumping stop only after typing any character.
  3. try to select this whole para and say apply a color. Color does not get applied to the last char in this paragraph.
  4. if we try to add another MC image to this para, the previous one gets replaced. Looks to be the same issue.
shrutimehta13 commented 6 years ago

reproducible on firefox also

connieimdialog commented 6 years ago

i can confirm this behaviour as well, which is problematic sometimes -- seems like i have to delete the whole paragraph below in order to get rid of the jumping. the only way that it seem to be prevented is if i make sure there i'm inserting the image amid a bunch of newline positions. this is why I'm confirming that the comment i just made in #8 @sapraaman

CrisNelson commented 6 years ago

Relates to #6 & #8

sapraaman commented 6 years ago

@shrutimehta13 can you please test once again on production and staging ? We deployed a manual fix on a 3rd party plugin, and that should have fixed it

shrutimehta13 commented 6 years ago

@sapraaman only Jumping behaviour resolved. Point 1 (MC not inserted at the cursor position) and Point 4 (MC replaced or adding a new MC search) are still pending