[TYPO3-core] RFC: #14717: [Usability] Add delay to close the context menu

Steffen Kamper info at sk-typo3.de
Mon Jun 14 13:15:27 CEST 2010


Hi,

Dmitry Dulepov schrieb:
> Hi!
> 
> Two notes.
> 
> Steffen Kamper wrote:
>> +		this.doNotHide = false;
> 
> What about a more speaking name, such as "delayClickMenuHide"? This is
> exactly what this variable is for. 'doNotHide' is indescriptive and prone
> to abuse.
>

i did a simple name because it's only used internally. But i can rename 
this var, no problem.


>> +		window.setTimeout(function() {
>> +			if (!Clickmenu.doNotHide) {
>> +				Element.hide(obj);
>> +			}
>> +		}, 500);
> 
> Why don't you use Ext.util.DelayedTask? Just a question.
> 

i don't want to mix a pure prototype script with ExtJS. As the 
contextmenu will be replaced with an ExtJS menu in 4.5 anyways i choosed 
this solution.

vg Steffen


More information about the TYPO3-team-core mailing list