Fix #2641: Input field change to blank after double click F2 to rename the file - few user will encounter

[TomMalbran]

This adds a renaming flag, so that when F2 is pressed for the second time, the code does nothing, leaving the renaming as it was, which fixes issue #2641.

[julianasuh]

Assigned @gruehle

[peterflynn]

Would it be better to just check ProjectManager._isInRename()?

[TomMalbran]

I didn't saw that function before. It should work to fix this too.

[gruehle]

This should go at the top of the function, before the call to showInTree(). There is no real harm in calling showInTree(), but it is not necessary since the item will be visible if it is being renamed.

[TomMalbran]

Fixed

[gruehle]

Initial review complete. It works as advertised, but the _isInRename() check could be done earlier.

[gruehle]

The _isInRename() function is expecting a DOM element, but entry is a FileEntry. You'll need to get the selected DOM element here.

It may just be easiest to put the code back where it was... Sorry about that! :-)

[TomMalbran]

Done. It's ok. I did a fast test and it worked, so I submitted the change, but I might have been using the old version even after reloading the second window.

[gruehle]

Thanks! Merging.