summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2017-08-16 07:02:05 +0200
committerGerrit Code Review <noreply-gerritcodereview@google.com>2017-08-16 07:02:05 +0200
commit162b92323bdcb13baf619450d994f5045f1ee195 (patch)
tree62656a73a7ef12dce8e7c83a825cb6a267495278
parentMerge "Add a new PatchChunk class in imgdiff" (diff)
parentscreen_ui: Word-wrap menu headers. (diff)
downloadandroid_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar.gz
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar.bz2
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar.lz
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar.xz
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.tar.zst
android_bootable_recovery-162b92323bdcb13baf619450d994f5045f1ee195.zip
-rw-r--r--recovery.cpp9
-rw-r--r--screen_ui.cpp30
-rw-r--r--screen_ui.h3
3 files changed, 37 insertions, 5 deletions
diff --git a/recovery.cpp b/recovery.cpp
index 233e56246..c5ead451a 100644
--- a/recovery.cpp
+++ b/recovery.cpp
@@ -757,12 +757,13 @@ static bool wipe_data(Device* device) {
}
static bool prompt_and_wipe_data(Device* device) {
+ // Use a single string and let ScreenRecoveryUI handles the wrapping.
const char* const headers[] = {
- "Can't load Android system. Your data may be corrupt.",
- "If you continue to get this message, you may need to",
- "perform a factory data reset and erase all user data",
+ "Can't load Android system. Your data may be corrupt. "
+ "If you continue to get this message, you may need to "
+ "perform a factory data reset and erase all user data "
"stored on this device.",
- NULL
+ nullptr
};
const char* const items[] = {
"Try again",
diff --git a/screen_ui.cpp b/screen_ui.cpp
index 8f792f162..e056512bd 100644
--- a/screen_ui.cpp
+++ b/screen_ui.cpp
@@ -278,6 +278,34 @@ int ScreenRecoveryUI::DrawTextLines(int x, int y, const char* const* lines) cons
return offset;
}
+int ScreenRecoveryUI::DrawWrappedTextLines(int x, int y, const char* const* lines) const {
+ int offset = 0;
+ for (size_t i = 0; lines != nullptr && lines[i] != nullptr; ++i) {
+ // The line will be wrapped if it exceeds text_cols_.
+ std::string line(lines[i]);
+ size_t next_start = 0;
+ while (next_start < line.size()) {
+ std::string sub = line.substr(next_start, text_cols_ + 1);
+ if (sub.size() <= text_cols_) {
+ next_start += sub.size();
+ } else {
+ // Line too long and must be wrapped to text_cols_ columns.
+ size_t last_space = sub.find_last_of(" \t\n");
+ if (last_space == std::string::npos) {
+ // No space found, just draw as much as we can
+ sub.resize(text_cols_);
+ next_start += text_cols_;
+ } else {
+ sub.resize(last_space);
+ next_start += last_space + 1;
+ }
+ }
+ offset += DrawTextLine(x, y + offset, sub.c_str(), false);
+ }
+ }
+ return offset;
+}
+
static const char* REGULAR_HELP[] = {
"Use volume up/down and power.",
NULL
@@ -316,7 +344,7 @@ void ScreenRecoveryUI::draw_screen_locked() {
y += DrawTextLines(x, y, HasThreeButtons() ? REGULAR_HELP : LONG_PRESS_HELP);
SetColor(HEADER);
- y += DrawTextLines(x, y, menu_headers_);
+ y += DrawWrappedTextLines(x, y, menu_headers_);
SetColor(MENU);
y += DrawHorizontalRule(y) + 4;
diff --git a/screen_ui.h b/screen_ui.h
index 8402fac00..df7cc25b3 100644
--- a/screen_ui.h
+++ b/screen_ui.h
@@ -187,6 +187,9 @@ class ScreenRecoveryUI : public RecoveryUI {
virtual int DrawTextLine(int x, int y, const char* line, bool bold) const;
// Draws multiple text lines. Returns the offset it should be moving along Y-axis.
int DrawTextLines(int x, int y, const char* const* lines) const;
+ // Similar to DrawTextLines() to draw multiple text lines, but additionally wraps long lines.
+ // Returns the offset it should be moving along Y-axis.
+ int DrawWrappedTextLines(int x, int y, const char* const* lines) const;
};
#endif // RECOVERY_UI_H