Revised title bar truncation code & removed title widget shading (#1201)

* Revised title truncate & undo widget shading

* Revised title truncation & undo widget shading
This commit is contained in:
Mark Thompson 2023-06-27 21:34:25 -05:00 committed by GitHub
parent 76948cffa8
commit 39acaf846a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 17 deletions

View File

@ -110,12 +110,6 @@ StatusTray::StatusTray(Point pos)
set_focusable(false); set_focusable(false);
} }
void StatusTray::add_button(ImageButton* child) {
child->set_background((shading_) ? Color::dark_grey() : Color::darker_grey());
shading_ = !shading_;
add(child);
}
void StatusTray::add(Widget* child) { void StatusTray::add(Widget* child) {
width_ += child->parent_rect().width(); width_ += child->parent_rect().width();
add_child(child); add_child(child);
@ -142,7 +136,6 @@ void StatusTray::clear() {
child->set_parent(nullptr); child->set_parent(nullptr);
children_.clear(); children_.clear();
width_ = 0; width_ = 0;
shading_ = false;
set_parent_rect({pos_, {width_, height}}); set_parent_rect({pos_, {width_, height}});
set_dirty(); set_dirty();
} }
@ -223,17 +216,17 @@ void SystemStatusView::refresh() {
// NB: Order of insertion is the display order Left->Right. // NB: Order of insertion is the display order Left->Right.
// TODO: Might be better to support hide and only add once. // TODO: Might be better to support hide and only add once.
status_icons.clear(); status_icons.clear();
if (!pmem::ui_hide_camera()) status_icons.add_button(&button_camera); if (!pmem::ui_hide_camera()) status_icons.add(&button_camera);
if (!pmem::ui_hide_sleep()) status_icons.add_button(&button_sleep); if (!pmem::ui_hide_sleep()) status_icons.add(&button_sleep);
if (!pmem::ui_hide_stealth()) status_icons.add_button(&button_stealth); if (!pmem::ui_hide_stealth()) status_icons.add(&button_stealth);
if (!pmem::ui_hide_converter()) status_icons.add_button(&button_converter); if (!pmem::ui_hide_converter()) status_icons.add(&button_converter);
if (!pmem::ui_hide_bias_tee()) status_icons.add_button(&button_bias_tee); if (!pmem::ui_hide_bias_tee()) status_icons.add(&button_bias_tee);
if (!pmem::ui_hide_clock()) status_icons.add_button(&button_clock_status); if (!pmem::ui_hide_clock()) status_icons.add(&button_clock_status);
if (!pmem::ui_hide_mute()) status_icons.add_button(&button_mute); if (!pmem::ui_hide_mute()) status_icons.add(&button_mute);
// Display "Disable speaker" icon only if AK4951 Codec which has separate speaker/headphone control // Display "Disable speaker" icon only if AK4951 Codec which has separate speaker/headphone control
if (audio::speaker_disable_supported() && !pmem::ui_hide_speaker()) if (audio::speaker_disable_supported() && !pmem::ui_hide_speaker())
status_icons.add_button(&button_speaker); status_icons.add(&button_speaker);
if (!pmem::ui_hide_sd_card()) status_icons.add(&sd_card_status_view); if (!pmem::ui_hide_sd_card()) status_icons.add(&sd_card_status_view);
status_icons.update_layout(); status_icons.update_layout();
@ -305,7 +298,7 @@ void SystemStatusView::set_title(const std::string new_value) {
title.set(default_title); title.set(default_title);
} else { } else {
// Limit length of title string to prevent partial characters if too many StatusView icons // Limit length of title string to prevent partial characters if too many StatusView icons
size_t max_len = (240 - 16 - 7 - status_icons.width()) / 8; size_t max_len = (status_icons.parent_rect().left() - title.parent_rect().left()) / 8;
title.set(truncate(new_value, max_len)); title.set(truncate(new_value, max_len));
} }
} }

View File

@ -144,7 +144,6 @@ class StatusTray : public View {
// track of the right edge. // track of the right edge.
const Point pos_{}; const Point pos_{};
uint8_t width_{}; uint8_t width_{};
bool shading_{};
}; };
class SystemStatusView : public View { class SystemStatusView : public View {