From e1075e2e4e95a2be41a1bb527d3d5313892fe738 Mon Sep 17 00:00:00 2001 From: Jannik Date: Thu, 18 Aug 2016 18:32:39 +0200 Subject: [PATCH] Work on the first Feedback MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Das Design und Farbschema entspricht nicht dem PFA-Design > Die Toolbar hat die falsche Farbe ->Check > Der Navigation-Drawer ist weder im alten, noch im neuen Design. Hier müsst ihr euch eines aussuchen. So wie euer Drawer gerade aussieht, bin ich der Meinung, dass es einfacher ist, das neue Design zu übernehmen. Ihr findet eine Beispielapp hier: https://github.com/SecUSo/privacy-friendly-app-example (Hier könnt ihr euch das Farbfile im Code suchen und rauskopieren, dann passen sich viele Farben automatisch an) -> Check - Die Überschrift sollte nur Break Reminder sein, also ohne das Privacy Friendly -> Check - Bitte keinen Farbverlauf im Hintergrund, ich würde vorschlagen den Hintergrund einfach weiß zu machen. (-> Check) - Die Buttons sind nicht im PFA-Design. Es gibt Standarddesigns für Buttons, die ebenfalls aus der Example-App hervorgehen. -> Check - Der Bildschirm schaltet sich bei mir leider aus, auch wenn in den Einstellungen "Screen stays on" aktiviert ist -> Fehlt noch Zähler: - Es fehlt der Kreis, den ihr beschrieben hattet (siehe FUD) -> Angefangen -> Noch keine Lösung Settings: - Was ist mit "New message notification" gemeint? - Generel: Hier kann man nur das aktuell ausgewählte Profil ändern. Das ist aus Usability-Sicht sehr schlecht, daher sollte man dort das Profil auch auswählen könne. -> Beides noch nicht gelöst New Profile: - Hier müsst ihr mal mit der Anordnung schauen, wenn es im finalen Design ist, dass es ordentlich aussieht. -> Ist ja schon besser als bei der Abgabe - Die Länge für den Profile-Name solltet ihr beschränken. (Genereller Tipp: Usereingaben sollten gerade bei Titeln nie unbeschränkt sein) -> Muss noch gemacht werden --- .../main/res/drawable/arrow_black_left.png | Bin 0 -> 1088 bytes .../main/res/drawable/arrow_black_right.png | Bin 0 -> 1089 bytes app/src/main/res/drawable/button_normal.xml | 9 +++ app/src/main/res/drawable/side_nav_bar.xml | 8 +-- .../res/layout/activity_break_reminder.xml | 60 ++++++++++++++++++ .../res/layout/app_bar_break_reminder.xml | 6 +- .../res/layout/content_break_reminder.xml | 30 +++++++-- app/src/main/res/values/colors.xml | 12 +++- app/src/main/res/values/strings.xml | 2 +- 9 files changed, 114 insertions(+), 13 deletions(-) create mode 100644 app/src/main/res/drawable/arrow_black_left.png create mode 100644 app/src/main/res/drawable/arrow_black_right.png create mode 100644 app/src/main/res/drawable/button_normal.xml diff --git a/app/src/main/res/drawable/arrow_black_left.png b/app/src/main/res/drawable/arrow_black_left.png new file mode 100644 index 0000000000000000000000000000000000000000..6f22be674773d95ad0fe83c346d161918b197882 GIT binary patch literal 1088 zcmeAS@N?(olHy`uVBq!ia0vp^4Is?H3?#oinD`S&F%}28J29*~C-V}>VM%xNb!1@J z*w6hZkrgNx5a1Ky3KYizR8-{U5q!UCEJMi5(|ih=6DDxrd4E=UAyCPWHqK9Ur&2*ON3K>>)RFpD5kK+>mh4m&VF z^GkyKf*BZ@m|0la*f}`4xOw;l1cgP!q+}ITG_-XMO|9&mT;03^!eWw=(=xLP%IoSI z8k<|%I(jBgnL2IO>^XDi&0nx^(c&dbmn~nha@FcJYuBycuyxn&J$v^bIeGfr<*V0k z-M;_m$+MSl-hKT1?Z?mGfBydKOq}xvXf*3KR>mn z{+p4}!Y-j#^9|qq&$wfsd&fTX+26_21-PaL{bzst@4MLFWDWm4#_#I0@BBaZ`{|DQ zMNA6?zV}Oi-*5J=KD6y`aOu9;%G5q^r77qPy+ zdSXk)K{s8awIPKEpYeiZe~TIEMt%6o2P7w~bYAAlUG{on=EbJ>ydahLVu4B?ED!`5 z@Z(~SPq)~gJ?%c)Tf6>A)NfS)nz#SGGLUTNp8(Xc*d3(fqcBLv5>cQIi(HU#oBTn> z#aM&PfA6wvYLsLBBI%i8`Y#-p9p#>U-*#ZHeSVVj_Xf(fwH=%tDnm{ Hr-UW|O=fVS literal 0 HcmV?d00001 diff --git a/app/src/main/res/drawable/arrow_black_right.png b/app/src/main/res/drawable/arrow_black_right.png new file mode 100644 index 0000000000000000000000000000000000000000..76bfaced063ca4777855a4bc95196604a0c35ae8 GIT binary patch literal 1089 zcmeAS@N?(olHy`uVBq!ia0vp^4Is?H3?#oinD`S&F%}28J29*~C-V}>VM%xNb!1@J z*w6hZkrgNx5a1Ky3KYizR8-{U5q!UCEJMi5(|ih=6DDxrd4E=UAyCPWHqK9Ur&2*ON3K>>)RFpD5kK+>mh4m&VF z^GkyKf*BZ@m|0la*f}`4xOw;l1cgP!q+}ITG_-XMO|9&mT;03^!eWw=(=xLP%IoSI z8k<|%I(jBgnL2IO>^XDi&0nx^(c&dbmn~nha@FcJYuBycuyxn&J$v^bIeGfr<*V0k z-M;_m$+MSl-hKT1?Z?mGfBydKOq}xvXfZf0uO2b=SsfoM-=i9Jfj_@f%%*9(3wPB+-HT1_AEsqED*n}}xu~juEKkd4A2!rf;=C;+idxu-^S+6RWN@kT`q$hiA|gogI5yftt2->PHt8GXpgp zv)Y$)@aue_g6m#DBK-P?RBpZXk4}r3)E%C#yW@xGeX%RWV%xb-10_qlfyCDNA5*zk z*#lLCoCOhn4^E3car^LJcgKI*zoMeMFSGjZycV*C^QFIqXKf>QsPrqw_E+|fEC0-$ zwrHM!+>{de!zn-BtX9ku)jzoQAs3&r+V}XzXrce6J5)bjW}kTEd~Z|qyJCrrcO-xU z_ZxsRx;spld^mnkw{!kK?Kz(NKgsTx>RB&zXVQ=02h)zsXWOpwk7tMKcgya`o2MIc zeiefR*@1#gKxO}JcP#zzIrX1Hy5Z@1hV3eKzpO$Y*E4P3^rC&!^Zq0G%s?Rthwz^d zCHI`-mrb&7+0Wl7euwjv{lC&bix2cZubc-$f7c86F|;04{5Id;$rqH@Jzf1=);T3K F0RRX=eSQD{ literal 0 HcmV?d00001 diff --git a/app/src/main/res/drawable/button_normal.xml b/app/src/main/res/drawable/button_normal.xml new file mode 100644 index 0000000..0232012 --- /dev/null +++ b/app/src/main/res/drawable/button_normal.xml @@ -0,0 +1,9 @@ + + + + + + + + + \ No newline at end of file diff --git a/app/src/main/res/drawable/side_nav_bar.xml b/app/src/main/res/drawable/side_nav_bar.xml index 24610b9..3dc0e1a 100644 --- a/app/src/main/res/drawable/side_nav_bar.xml +++ b/app/src/main/res/drawable/side_nav_bar.xml @@ -1,9 +1,9 @@ \ No newline at end of file diff --git a/app/src/main/res/layout/activity_break_reminder.xml b/app/src/main/res/layout/activity_break_reminder.xml index c574778..b0aebed 100644 --- a/app/src/main/res/layout/activity_break_reminder.xml +++ b/app/src/main/res/layout/activity_break_reminder.xml @@ -1,4 +1,9 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/app/src/main/res/layout/app_bar_break_reminder.xml b/app/src/main/res/layout/app_bar_break_reminder.xml index 9079b2c..98e0076 100644 --- a/app/src/main/res/layout/app_bar_break_reminder.xml +++ b/app/src/main/res/layout/app_bar_break_reminder.xml @@ -17,10 +17,14 @@ android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" android:background="?attr/colorPrimary" - app:popupTheme="@style/AppTheme.PopupOverlay" /> + app:popupTheme="@style/AppTheme.PopupOverlay"> + + + + diff --git a/app/src/main/res/layout/content_break_reminder.xml b/app/src/main/res/layout/content_break_reminder.xml index d38e1c1..0845741 100644 --- a/app/src/main/res/layout/content_break_reminder.xml +++ b/app/src/main/res/layout/content_break_reminder.xml @@ -9,6 +9,7 @@ android:paddingRight="@dimen/activity_horizontal_margin" android:paddingTop="@dimen/activity_vertical_margin" app:layout_behavior="@string/appbar_scrolling_view_behavior" + android:background="@color/middleblue" tools:context="orgprivacy_friendly_apps.secuso.privacyfriendlybreakreminder.BreakReminder" tools:showIn="@layout/app_bar_break_reminder"> @@ -26,15 +27,18 @@ android:popupBackground="@drawable/side_nav_bar" android:spinnerMode="dropdown" /> +