Skip to content

Commit

Permalink
Disable double-click -> ESC translation for main menu
Browse files Browse the repository at this point in the history
  • Loading branch information
Zeno- committed Mar 14, 2015
1 parent 0e93eef commit de6d5ce
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 12 deletions.
3 changes: 2 additions & 1 deletion src/guiEngine.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,8 @@ GUIEngine::GUIEngine( irr::IrrlichtDevice* dev,
m_texture_source,
m_formspecgui,
m_buttonhandler,
NULL);
NULL,
false);

m_menu->allowClose(false);
m_menu->lockSize(true,v2u32(800,600));
Expand Down
22 changes: 13 additions & 9 deletions src/guiFormSpecMenu.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ GUIFormSpecMenu::GUIFormSpecMenu(irr::IrrlichtDevice* dev,
gui::IGUIElement* parent, s32 id, IMenuManager *menumgr,
InventoryManager *invmgr, IGameDef *gamedef,
ISimpleTextureSource *tsrc, IFormSource* fsrc, TextDest* tdst,
Client* client) :
Client* client, bool remap_dbl_click) :
GUIModalMenu(dev->getGUIEnvironment(), parent, id, menumgr),
m_device(dev),
m_invmgr(invmgr),
Expand All @@ -97,7 +97,8 @@ GUIFormSpecMenu::GUIFormSpecMenu(irr::IrrlichtDevice* dev,
m_text_dst(tdst),
m_formspec_version(0),
m_focused_element(L""),
m_font(NULL)
m_font(NULL),
m_remap_dbl_click(remap_dbl_click)
#ifdef __ANDROID__
,m_JavaDialogFieldName(L"")
#endif
Expand Down Expand Up @@ -2939,15 +2940,18 @@ bool GUIFormSpecMenu::preprocessEvent(const SEvent& event)
bool GUIFormSpecMenu::DoubleClickDetection(const SEvent event)
{
/* The following code is for capturing double-clicks of the mouse button
* that are *not* in/in a control (i.e. when the mouse if positioned in an
* unused area of the formspec) and translating the double-click into an
* EET_KEY_INPUT_EVENT event which closes the form.
* and translating the double-click into an EET_KEY_INPUT_EVENT event
* -- which closes the form -- under some circumstances.
*
* There have been many github issues reporting this as a bug even though it
* was an intended feature. For this reason the code has been disabled for
* non-Android builds
* was an intended feature. For this reason, remapping the double-click as
* an ESC must be explicitly set when creating this class via the
* /p remap_dbl_click parameter of the constructor.
*/
#ifdef __ANDROID__

if (!m_remap_dbl_click)
return false;

if (event.MouseInput.Event == EMIE_LMOUSE_PRESSED_DOWN) {
m_doubleclickdetect[0].pos = m_doubleclickdetect[1].pos;
m_doubleclickdetect[0].time = m_doubleclickdetect[1].time;
Expand Down Expand Up @@ -2986,7 +2990,7 @@ bool GUIFormSpecMenu::DoubleClickDetection(const SEvent event)
delete translated;
return true;
}
#endif

return false;
}

Expand Down
12 changes: 10 additions & 2 deletions src/guiFormSpecMenu.h
Original file line number Diff line number Diff line change
Expand Up @@ -210,8 +210,8 @@ class GUIFormSpecMenu : public GUIModalMenu
ISimpleTextureSource *tsrc,
IFormSource* fs_src,
TextDest* txt_dst,
Client* client
);
Client* client,
bool remap_dbl_click = true);

~GUIFormSpecMenu();

Expand Down Expand Up @@ -436,6 +436,14 @@ class GUIFormSpecMenu : public GUIModalMenu
std::wstring m_JavaDialogFieldName;
#endif

/* If true, remap a double-click (or double-tap) action to ESC. This is so
* that, for example, Android users can double-tap to close a formspec.
*
* This value can (currently) only be set by the class constructor
* and the default value for the setting is true.
*/
bool m_remap_dbl_click;

};

class FormspecFormSource: public IFormSource
Expand Down

0 comments on commit de6d5ce

Please sign in to comment.