Remove TWRP app prompt from reboot flow
Add option to install TWRP app as a list menu item on the
advanced page and as a button on the reboot page (except for
watch theme). Now when a user reboots the device, they no longer
will be prompted to install the TWRP app, which interrupts the
flow of rebooting the device.
Change-Id: I00559d0be6e7d0b00d469a5960e4b14e32a3fc5f
diff --git a/gui/theme/common/landscape.xml b/gui/theme/common/landscape.xml
index 23e5338..ba6073e 100755
--- a/gui/theme/common/landscape.xml
+++ b/gui/theme/common/landscape.xml
@@ -1015,7 +1015,7 @@
<page name="rebootcheck">
<action>
<condition var1="tw_backup_system_size" op=">=" var2="%tw_min_system%"/>
- <action function="page">appcheck</action>
+ <action function="reboot">%tw_reboot_param%</action>
</action>
<action>
@@ -1024,28 +1024,6 @@
</action>
</page>
- <page name="appcheck">
- <action>
- <condition var1="tw_app_prompt" var2="1"/>
- <action function="checkforapp"></action>
- </action>
-
- <action>
- <condition var1="tw_app_prompt" op="!=" var2="1"/>
- <action function="reboot">%tw_reboot_param%</action>
- </action>
-
- <action>
- <condition var1="tw_app_install_status" var2="1"/>
- <action function="page">rebootapp</action>
- </action>
-
- <action>
- <condition var1="tw_app_install_status" var2="2"/>
- <action function="reboot">%tw_reboot_param%</action>
- </action>
- </page>
-
<page name="wipe">
<template name="page"/>
@@ -2846,6 +2824,13 @@
</page>
<page name="reboot">
+ <action>
+ <action function="checkforapp"></action>
+ <action function="page">reboot2</action>
+ </action>
+ </page>
+
+ <page name="reboot2">
<template name="page"/>
<text style="text_l">
@@ -2968,9 +2953,20 @@
</button>
</listbox>
+ <button style="main_button">
+ <condition var1="tw_app_install_status" var2="1"/>
+ <placement x="%center_x%" y="%row12_y%"/>
+ <text>{@reboot_install_app_hdr=Install TWRP App}</text>
+ <actions>
+ <action function="set">tw_back=reboot</action>
+ <action function="set">tw_appinstall_title={@reboot_hdr=Reboot}</action>
+ <action function="page">installapp</action>
+ </actions>
+ </button>
+
<text style="text_m">
<condition var1="tw_has_boot_slots" var2="1"/>
- <placement x="%center_x%" y="%row14_y%" placement="5"/>
+ <placement x="%col1_x_header%" y="%row14_y%"/>
<text>{@current_boot_slot=Current Slot: %tw_active_slot%}</text>
</text>
@@ -3017,17 +3013,17 @@
</action>
</page>
- <page name="rebootapp">
+ <page name="installapp">
<template name="page"/>
<text style="text_l">
<placement x="%col1_x_header%" y="%row3_header_y%"/>
- <text>{@reboot_hdr=Reboot}</text>
+ <text>%tw_appinstall_title%</text>
</text>
<text style="text_m">
<placement x="%col1_x_header%" y="%row4_header_y%"/>
- <text>{@reboot_install_app_hdr=Install TWRP App?}</text>
+ <text>{@reboot_install_app_hdr=Install TWRP App}</text>
</text>
<text style="text_m_accent">
@@ -3041,12 +3037,6 @@
</text>
<checkbox>
- <placement x="%indent%" y="%row5_y%"/>
- <text>{@reboot_install_app_prompt_install=Prompt to install TWRP app if not installed}</text>
- <data variable="tw_app_prompt"/>
- </checkbox>
-
- <checkbox>
<condition var1="tw_mount_system_ro" var2="0"/>
<placement x="%indent%" y="%row7_y%"/>
<text>{@reboot_install_app_system=Install as a System App}</text>
@@ -3056,20 +3046,16 @@
<button style="main_button_half_width">
<placement x="%col2_x_left%" y="%row15a_y%"/>
<text>{@install_cancel=Do not Install}</text>
- <action function="reboot">%tw_reboot_param%</action>
+ <action function="page">%tw_back%</action>
</button>
<slider>
<text>{@swipe_to_install_app=Swipe to Install TWRP App}</text>
<actions>
- <action function="set">tw_back=reboot</action>
<action function="set">tw_action=installapp</action>
<action function="set">tw_action_text1={@reboot_installing_app=Installing App...}</action>
<action function="set">tw_action_text2=</action>
- <action function="set">tw_has_action2=1</action>
- <action function="set">tw_action2=reboot</action>
- <action function="set">tw_action2_param=%tw_reboot_param%</action>
- <action function="set">tw_complete_text1={@rebooting=Rebooting...}</action>
+ <action function="set">tw_complete_text1={@successful=Successful}</action>
<action function="page">action_page</action>
</actions>
</slider>
@@ -3086,7 +3072,7 @@
<touch key="back"/>
<actions>
<action function="set">tw_app_install_status=0</action>
- <action function="page">reboot</action>
+ <action function="page">%tw_back%</action>
</actions>
</action>
</page>
@@ -3195,9 +3181,6 @@
<listitem name="{@use24clock_chk=Use 24-hour clock}">
<data variable="tw_military_time"/>
</listitem>
- <listitem name="{@reboot_install_app_prompt_install=Prompt to install TWRP app on every reboot}">
- <data variable="tw_app_prompt"/>
- </listitem>
<listitem name="{@simact_chk=Simulate actions for theme testing}">
<data variable="tw_simulate_actions"/>
</listitem>
@@ -3610,6 +3593,13 @@
</page>
<page name="advanced">
+ <action>
+ <action function="checkforapp"></action>
+ <action function="page">advanced1</action>
+ </action>
+ </page>
+
+ <page name="advanced1">
<template name="page"/>
<text style="text_l">
@@ -3718,6 +3708,14 @@
<action function="page">confirm_action</action>
</actions>
</listitem>
+ <listitem name="{@reboot_install_app_hdr=Install TWRP App}">
+ <condition var1="tw_app_install_status" var2="1"/>
+ <actions>
+ <action function="set">tw_back=advanced</action>
+ <action function="set">tw_appinstall_title={@advanced_hdr=Advanced}</action>
+ <action function="page">installapp</action>
+ </actions>
+ </listitem>
<listitem name="{@uninstall_twrp_system_app=Uninstall TWRP App from System}">
<condition var1="tw_app_installed_in_system" var2="1"/>
<actions>