#2252: Setting to disable our custom menu
--------------------------+-------------------------------------------------
Reporter: fredck | Owner: w.olchawa
Type: New Feature | Status: new
Priority: Normal | Milestone: FCKeditor 2.6.1
Component: General | Version:
Resolution: | Keywords: Confirmed Review-
--------------------------+-------------------------------------------------
Changes (by martinkou):
* keywords: Confirmed Review+ => Confirmed Review-
Comment:
Unfortunately, I have to reverse my Review+ because of some potential
problems in the patch.
First problem... the patch causes the browser menu to appear automagically
if the user upgraded from a previous FCKeditor version's fckconfig.js -
FCKConfig.BrowserContextMenu == false evaluates to false when the option
is undefined.
Second problem... the current approach doesn't allow the setting to be
changed after FCKeditor has loaded. It might be better if the check is
implemented at the context menu/click event handler, instead of at load
time.
Much thanks to Alfonso for reporting these problems to me in the IRC
channel.
--
Ticket URL: <https://dev.fckeditor.net/ticket/2252#comment:5>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
