[wp-trac] [WordPress Trac] #43012: Cannot Update Post in Firefox Due to Editor and TinyMCE JavaScript TypeErrors

WordPress Trac noreply at wordpress.org
Tue Jan 2 22:55:06 UTC 2018


#43012: Cannot Update Post in Firefox Due to Editor and TinyMCE JavaScript
TypeErrors
-------------------------------+------------------------------
 Reporter:  johnschulz         |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Editor             |     Version:  4.9.1
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |     Focuses:
-------------------------------+------------------------------
Changes (by azaozz):

 * keywords:   => reporter-feedback


Comment:

 Hi @johnschulz, thanks for the bug report.

 This seems to be the actual (non-minified) code you're referencing:
 https://core.trac.wordpress.org/browser/tags/4.9.1/src/wp-
 admin/js/editor.js#L697.

 I don't yet have Firefox 58, 57.0.3 seems to be the latest stable and all
 seems to be working properly there. In the `var b=a.getWin()`, `a` is the
 editor instance and there is a check if it exists before running that
 code. Also the function `findBookmarkedPosition()` (that contains this
 code) runs only when switching the editor from Visual to Text. It doesn't
 run on saving/updating the post.

 It is possible that there is a change (or even a bug) in the way
 `window.getSelection()` works in Firefox 58, although my guess is that
 there is something else at play here. Do you have any WordPress plugins or
 Firefox add-ons installed?

 You can also try to debug this further by adding `define( 'SCRIPT_DEBUG',
 true );` to wp-config.php. That will load the non-minified scripts, see:
 https://codex.wordpress.org/Debugging_in_WordPress#SCRIPT_DEBUG for more
 info.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/43012#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list