#11343: TinyMCE won't work with @ in the username of notebook
------------------------+---------------------------------------------------
Reporter: kcrisman | Owner: jason, mpatel, was
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-4.7.2
Component: notebook | Keywords: notebook, tinymce, at, sd31
Work_issues: | Upstream: Reported upstream. Little or no
feedback.
Reviewer: | Author: Karl-Dieter Crisman
Merged: | Dependencies:
------------------------+---------------------------------------------------
Changes (by newvalueoldvalue):
* priority: major => critical
* status: needs_info => needs_review
* author: => Karl-Dieter Crisman
Old description:
> Please see below for what TinyMCE cells look like if you have an `@` in
> your username. This is since at least 4.6, see
> [http://groups.google.com/group/sage-
> notebook/browse_thread/thread/f0ceaa24700f4059 this thread].
>
> I'm making this critical, because it removes major piece of
> functionality. At the '''very least''', we'll need to change the login
> page to remove the @ availability!!!
>
> From #8995, a dup of this:
>
> See the thread by Dennis Watson here:
>
> http://groups.google.com/group/sage-
> support/browse_frm/thread/2acd499a566efce1
>
> In particular, some tinymce javascript files were trying to download from
> a URL that included the username, like:
>
> http://sagenb.org/home/usernamewith@/javascript/tiny_mce/langs/en.js
New description:
Please see below for what TinyMCE cells look like if you have an `@` in
your username. This is since at least 4.6, see
[http://groups.google.com/group/sage-
notebook/browse_thread/thread/f0ceaa24700f4059 this thread].
I'm making this critical, because it removes major piece of functionality.
At the '''very least''', we'll need to change the login page to remove the
@ availability!!!
Re-enabling this is #11470.
----
From #8995, a dup of this:
See the thread by Dennis Watson here:
http://groups.google.com/group/sage-
support/browse_frm/thread/2acd499a566efce1
In particular, some tinymce javascript files were trying to download from
a URL that included the username, like:
http://sagenb.org/home/usernamewith@/javascript/tiny_mce/langs/en.js
--
Comment:
Replying to [comment:24 jdemeyer]:
> This ticket is listed as blocker, what is the current status? It seems
like disallowing users to register usernames with @ could be a quick and
eays solution. But they should still be able to ''log in'' with any
existing username.
Which is precisely what [attachment:trac_11343-fix_at_symbol.patch] does.
I just checked and it still applies cleanly... I just feel this is better
than waiting eternally for the flask or TinyMCE update.
I really disagree that it's major and not blocker. Sage doesn't work
properly, and saying "use another browser" is going to keep on being said
until we have zero browsers that support it. If ppurka finds it doesn't
work with FF or Opera with the new TinyMCE, and it doesn't work on Safari,
and we already suggest (though nowhere official!!!) that IE is gone...
should we be sending people to lynx?
So critical. But if someone just reviews this patch, it fixes the
problem, and that is better. Plus it lights a fire under the flask/sagenb
people to finish things off so we don't mess with things any more.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11343#comment:26>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
