From e4f3f4b1346358340a66770f099fe7e18ec3f87c Mon Sep 17 00:00:00 2001 From: Oskar Chodowicz Date: Fri, 15 Feb 2019 19:22:56 +0100 Subject: [PATCH 01/16] fix for crash on switch addition Change-Id: Ibd89c8e70c827bad18e499f4795f7d26b2e876d4 --- src/UniversalSwitchPage.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/UniversalSwitchPage.cpp b/src/UniversalSwitchPage.cpp index 15c7be9..3732250 100644 --- a/src/UniversalSwitchPage.cpp +++ b/src/UniversalSwitchPage.cpp @@ -138,7 +138,8 @@ void UniversalSwitchPage::onNewSwitchAddition(void *data, const std::string &swi { retm_if(data == NULL, "Input parameter is NULL"); auto ad = static_cast(data); - SwitchesPage::addSwitchCb(ad, switch_id, switch_action, switch_name); + //TODO Try to use service which is independent from specific view + setting_accessibility_universal_switch_dbus_config_addSwitchConfigurationItem(&ad->config, switch_id.c_str(), switch_name.c_str(), switch_action.c_str()); int switch_count = setting_accessibility_universal_switch_dbus_config_get_switch_count(&ad->config); if (switch_count == 1) -- 2.7.4 From ca1746d531c3f787953b95457f9219afc28a123d Mon Sep 17 00:00:00 2001 From: Oskar Chodowicz Date: Wed, 6 Feb 2019 12:43:44 +0100 Subject: [PATCH 02/16] Set all widgets defaultly visible Change-Id: I8d4f391015c985db8ec4d07042107697e2030781 --- src/AccessoriesSwitchesPage.cpp | 10 ---------- src/Box.cpp | 1 + src/Button.cpp | 2 ++ src/Check.cpp | 1 + src/Conformant.cpp | 1 + src/CtxPopup.cpp | 1 + src/Entry.cpp | 1 + src/Genlist.cpp | 1 + src/Icon.cpp | 1 + src/Image.cpp | 1 + src/Layout.cpp | 1 + src/MainData.cpp | 5 ----- src/Naviframe.cpp | 1 + src/Popup.cpp | 2 ++ src/Radio.cpp | 1 + src/RemoveSwitchPage.cpp | 2 -- src/ScreenReaderSettingsPage.cpp | 5 ----- src/Scroller.cpp | 1 + src/SetValuePage.cpp | 2 -- src/Slider.cpp | 1 + src/Toolbar.cpp | 1 + src/UniversalSwitchPage.cpp | 3 --- src/UpdateSwitchPage.cpp | 2 -- src/Widget.cpp | 5 +++++ src/Widget.hpp | 1 + 25 files changed, 24 insertions(+), 29 deletions(-) diff --git a/src/AccessoriesSwitchesPage.cpp b/src/AccessoriesSwitchesPage.cpp index 47d414e..f833c25 100644 --- a/src/AccessoriesSwitchesPage.cpp +++ b/src/AccessoriesSwitchesPage.cpp @@ -28,7 +28,6 @@ AccessoriesSwitchesPage::AccessoriesSwitchesPage(SettingAccessibility *ad) auto naviframe = context_->md.getNaviframe(); auto layout = Widget::make(naviframe, EDJ_ACCESSORY, GRP_ACCESSORY); layout->setText(PRT_ACCESSORY_LABEL, "IDS_ACCS_UNIVERSAL_SWITCH_ADD_SWITCH_ACCESSORIES_DESC"); - layout->show(); naviframe->pushBack("IDS_ACCS_UNIVERSAL_SWITCH_ADD_SWITCH_ACCESSORIES_TITLE", layout, {}, [this]() { setting_accessibility_universal_switch_dbus_config_cancelCaptureSwitch(&context_->config); @@ -39,7 +38,6 @@ AccessoriesSwitchesPage::AccessoriesSwitchesPage(SettingAccessibility *ad) naviframe->popBack(); }); prevBtn->setStyle(Button::BACK_BUTTON_ARROW_STYLE); - prevBtn->show(); layout->setPartContent("prev_btn", prevBtn); context_->us_configuration.switch_id = {}; @@ -74,13 +72,11 @@ void AccessoriesSwitchesPage::newSwitchPopup() popup->setEextEventCallback(EEXT_CALLBACK_BACK, []() {}); auto layout = Widget::make(popup, EDJ_ACCESSORY_POPUP, GRP_ACCESSORY_POPUP); - layout->show(); popup->setContent(layout); auto entry = Widget::make(layout); entry->setWeightHint(EVAS_HINT_EXPAND, EVAS_HINT_EXPAND); entry->setPartText("guide", "IDS_ACCS_UNIVERSAL_SWITCH_ADD_SWITCH_NAME"); - entry->show(); layout->setPartContent(PRT_ACCESSORY_POPUP_ENTRY, entry); auto cancelBtnCb = [popup, this]() { @@ -91,7 +87,6 @@ void AccessoriesSwitchesPage::newSwitchPopup() auto cancelBtn = Widget::make