Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: mobile/android/app/src/photon/res/values/styles.xml

Issue 29986555: Issue 7200 - Adjust UI colors (Closed)
Patch Set: Created Jan. 21, 2019, 6:56 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mobile/android/app/src/photon/res/values/styles.xml
===================================================================
--- a/mobile/android/app/src/photon/res/values/styles.xml
+++ b/mobile/android/app/src/photon/res/values/styles.xml
@@ -356,18 +356,17 @@
<item name="android:textColorHint">?android:attr/textColorHintInverse</item>
<item name="android:textColorHighlight">@color/text_color_highlight_inverse</item>
<item name="android:textColorLink">?android:attr/textColorLink</item>
</style>
<style name="TextAppearance.EmptyMessage" parent="TextAppearance.Large"/>
<style name="TextAppearance.EmptyHint" parent="TextAppearance.Medium">
- <!-- Using ABB color palette. See https://issues.adblockplus.org/ticket/3769 -->
- <item name="android:textColor">@color/abb_accent_dark_blue</item>
+ <item name="android:textColor">#FFA62F</item>
anton 2019/01/28 05:53:03 shouldn't it be in @color/.. ?
diegocarloslima 2019/01/28 11:43:08 Yeap, but I basically reverted this to the upstrea
<item name="android:textStyle">italic</item>
</style>
<style name="TextAppearance.Micro">
<item name="android:textSize">12sp</item>
<item name="android:textColor">?android:attr/textColorTertiary</item>
</style>
@@ -408,18 +407,17 @@
<item name="android:textSize">12sp</item>
</style>
<style name="TextAppearance.Widget.Home.ItemTitle" parent="TextAppearance">
<item name="android:textSize">16sp</item>
</style>
<style name="TextAppearance.Widget.Home.ItemDescription" parent="TextAppearance.Micro">
- <!-- Using ABB color palette. See https://issues.adblockplus.org/ticket/3769 -->
- <item name="android:textColor">@color/abb_text_divider_grey</item>
+ <item name="android:textColor">@color/tabs_tray_icon_grey</item>
</style>
<style name="TextAppearance.Widget.HomeBanner" parent="TextAppearance.Small">
<item name="android:textColor">?android:attr/textColorHint</item>
</style>
<style name="TextAppearance.DoorHanger">
<item name="android:textColor">@color/placeholder_active_grey</item>
@@ -553,18 +551,17 @@
<!-- TabsLayout RemoteTabs Row Url -->
<style name="TabLayoutItemTextAppearance.Url">
<item name="android:textColor">#FFA4A7A9</item>
</style>
<!-- Find bar -->
<style name="FindBar">
- <!-- Using ABB color palette. See https://issues.adblockplus.org/ticket/3769 -->
- <item name="android:background">@color/abb_background_light_grey</item>
+ <item name="android:background">@color/text_and_tabs_tray_grey</item>
<item name="android:paddingLeft">3dip</item>
<item name="android:paddingStart">3dip</item>
<item name="android:paddingRight">3dip</item>
<item name="android:paddingEnd">3dip</item>
<item name="android:paddingTop">6dip</item>
<item name="android:paddingBottom">6dip</item>
</style>
« no previous file with comments | « mobile/android/app/src/main/res/values/themes.xml ('k') | mobile/android/base/java/org/mozilla/gecko/tabs/TabStrip.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld