From ee482ae3a0800f155a5302f5b990a05294ee2baf Mon Sep 17 00:00:00 2001 From: Nebuleon Fumika Date: Tue, 7 Oct 2014 04:31:17 +0000 Subject: [PATCH] Get rid of the action on D-pad Left in BrowseDialog There is no counterpart action for the opposite direction, D-pad Right. Having D-pad Left trigger a return to the parent directory was inconsistent. Additionally, this frees up the D-pad Left and Right inputs for something more useful. --- src/browsedialog.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/browsedialog.cpp b/src/browsedialog.cpp index 3d357f6..7e38bfc 100644 --- a/src/browsedialog.cpp +++ b/src/browsedialog.cpp @@ -31,11 +31,9 @@ void BrowseDialog::initButtonBox() { buttonBox.clear(); - // Left or Cancel to go up a directory, if directories are shown. + // Cancel to go up a directory, if directories are shown. // Accept also goes up a directory if the selection is "..". if (fl.getShowDirectories() && getPath() != "/") { - buttonBox.add(unique_ptr(new IconButton( - gmenu2x, "skin:imgs/buttons/left.png"))); if (selected < fl.size() && fl[selected] == "..") { buttonBox.add(unique_ptr(new IconButton( gmenu2x, "skin:imgs/buttons/accept.png"))); @@ -161,7 +159,6 @@ BrowseDialog::Action BrowseDialog::getAction(InputManager::Button button) return BrowseDialog::ACT_SCROLLUP; case InputManager::ALTRIGHT: return BrowseDialog::ACT_SCROLLDOWN; - case InputManager::LEFT: case InputManager::CANCEL: return BrowseDialog::ACT_GOUP; case InputManager::ACCEPT: