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/portrait.xml b/gui/theme/common/portrait.xml
index 9208a57..e5be1e3 100755
--- a/gui/theme/common/portrait.xml
+++ b/gui/theme/common/portrait.xml
@@ -966,7 +966,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>
@@ -975,28 +975,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"/>
@@ -3003,6 +2981,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">
@@ -3162,6 +3147,17 @@
</actions>
</button>
+ <button style="main_button_half_height">
+ <condition var1="tw_app_install_status" var2="1"/>
+ <placement x="%indent%" y="%row22_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>
+
<action>
<touch key="home"/>
<action function="page">main</action>
@@ -3173,17 +3169,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">
@@ -3197,12 +3193,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>
@@ -3212,20 +3202,16 @@
<button style="main_button_half_height">
<placement x="%indent%" y="%row16_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>
@@ -3242,7 +3228,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>
@@ -3369,9 +3355,6 @@
<listitem name="{@rev_navbar_chk=Reversed navbar layout}">
<data variable="tw_samsung_navbar"/>
</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>
@@ -3731,6 +3714,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">
@@ -3840,6 +3830,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>