From 45d770b1319d0fbf26e6b9dfe191550ec7340fa0 Mon Sep 17 00:00:00 2001
From: hius07 <62179190+hius07@users.noreply.github.com>
Date: Fri, 23 Jul 2021 18:19:36 +0300
Subject: [PATCH] Activate menu: standardize to 2 checkboxes (#8017)

---
 frontend/ui/elements/menu_activate.lua | 45 ++++++++------------------
 1 file changed, 13 insertions(+), 32 deletions(-)

diff --git a/frontend/ui/elements/menu_activate.lua b/frontend/ui/elements/menu_activate.lua
index f4df40df3..18259a354 100644
--- a/frontend/ui/elements/menu_activate.lua
+++ b/frontend/ui/elements/menu_activate.lua
@@ -2,55 +2,36 @@ local InfoMessage = require("ui/widget/infomessage")
 local UIManager = require("ui/uimanager")
 local _ = require("gettext")
 
-local function activateMenu() return G_reader_settings:readSetting("activate_menu") end
-
 return {
     text = _("Activate menu"),
     sub_item_table = {
         {
-            text = _("Swipe and tap"),
+            text = _("Tap"),
             checked_func = function()
-                local activate_menu = activateMenu()
-                if activate_menu == nil or activate_menu == "swipe_tap" then
-                    return true
-                else
-                    return false
-                end
+                return G_reader_settings:readSetting("activate_menu") ~= "swipe"
             end,
             callback = function()
-                G_reader_settings:saveSetting("activate_menu", "swipe_tap")
-                UIManager:show(InfoMessage:new{
-                    text = _("This will take effect on next restart."),
-                })
-            end
-        },
-        {
-            text = _("Only swipe"),
-            checked_func = function()
-                if activateMenu() == "swipe" then
-                    return true
+                if G_reader_settings:readSetting("activate_menu") ~= "swipe" then
+                    G_reader_settings:saveSetting("activate_menu", "swipe")
                 else
-                    return false
+                    G_reader_settings:saveSetting("activate_menu", "swipe_tap")
                 end
-            end,
-            callback = function()
-                G_reader_settings:saveSetting("activate_menu", "swipe")
                 UIManager:show(InfoMessage:new{
                     text = _("This will take effect on next restart."),
                 })
-            end
+            end,
         },
         {
-            text = _("Only tap"),
+            text = _("Swipe"),
             checked_func = function()
-                if activateMenu() == "tap" then
-                    return true
-                else
-                    return false
-                end
+                return G_reader_settings:readSetting("activate_menu") ~= "tap"
             end,
             callback = function()
-                G_reader_settings:saveSetting("activate_menu", "tap")
+                if G_reader_settings:readSetting("activate_menu") ~= "tap" then
+                    G_reader_settings:saveSetting("activate_menu", "tap")
+                else
+                    G_reader_settings:saveSetting("activate_menu", "swipe_tap")
+                end
                 UIManager:show(InfoMessage:new{
                     text = _("This will take effect on next restart."),
                 })