Fix Edit Bookmark and Authentication dialog theming (#793)

* added new style for TextInputLayout

* authentication dialog should also match the proper theming

Author: malmstein

app/src/main/res/layout/edit_bookmark.xml

@@ -20,18 +20,18 @@
     android:layout_height="match_parent"
     android:paddingTop="20dp">
 
-
     <com.google.android.material.textfield.TextInputLayout
         android:id="@+id/bookmarkTitleInputContainer"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
+        android:layout_height="60dp"
         android:layout_marginStart="20dp"
         android:layout_marginEnd="20dp"
+        style="@style/TextInputLayoutTheme"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent">
 
-        <EditText
+        <com.google.android.material.textfield.TextInputEditText
             android:id="@+id/titleInput"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
@@ -43,12 +43,13 @@
     <com.google.android.material.textfield.TextInputLayout
         android:id="@+id/bookmarkUrlInputContainer"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
+        android:layout_height="60dp"
         android:layout_marginStart="20dp"
         android:layout_marginEnd="20dp"
+        style="@style/TextInputLayoutTheme"
         app:layout_constraintTop_toBottomOf="@id/bookmarkTitleInputContainer">
 
-        <EditText
+        <com.google.android.material.textfield.TextInputEditText
             android:id="@+id/urlInput"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"

app/src/main/res/layout/http_authentication.xml

@@ -15,58 +15,60 @@
   -->
 
 <androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
-                                                   xmlns:app="http://schemas.android.com/apk/res-auto"
-                                                   android:layout_width="match_parent"
-                                                   android:layout_height="match_parent" xmlns:tools="http://schemas.android.com/tools"
-                                                   android:paddingTop="20dp">
-
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    android:paddingTop="20dp">
 
     <TextView
         android:id="@+id/httpAuthInformationText"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_marginTop="8dp"
         android:layout_marginStart="20dp"
+        android:layout_marginTop="8dp"
         android:layout_marginEnd="20dp"
         app:layout_constraintTop_toTopOf="parent" />
 
     <com.google.android.material.textfield.TextInputLayout
         android:id="@+id/httpAuthUsernameContainer"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
+        android:layout_height="60dp"
+        style="@style/TextInputLayoutTheme"
         android:layout_marginStart="20dp"
-        android:layout_marginEnd="20dp"
         android:layout_marginTop="20dp"
+        android:layout_marginEnd="20dp"
         app:layout_constraintTop_toBottomOf="@id/httpAuthInformationText">
 
-        <EditText
+        <com.google.android.material.textfield.TextInputEditText
             android:id="@+id/usernameInput"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:autofillHints="username"
             android:hint="@string/authenticationDialogUsernameHint"
-            android:inputType="text"
             android:importantForAutofill="yes"
-            android:autofillHints="username"
+            android:inputType="text"
             tools:ignore="UnusedAttribute" />
 
     </com.google.android.material.textfield.TextInputLayout>
 
     <com.google.android.material.textfield.TextInputLayout
         android:id="@+id/httpAuthPasswordContainer"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
+        android:layout_height="60dp"
+        style="@style/TextInputLayoutTheme"
         android:layout_marginStart="20dp"
         android:layout_marginEnd="20dp"
         app:layout_constraintTop_toBottomOf="@id/httpAuthUsernameContainer">
 
-        <EditText
+        <com.google.android.material.textfield.TextInputEditText
             android:id="@+id/passwordInput"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:autofillHints="password"
             android:hint="@string/authenticationDialogPasswordHint"
-            android:inputType="textWebPassword"
             android:importantForAutofill="yes"
-            android:autofillHints="password"
+            android:inputType="textWebPassword"
             tools:ignore="UnusedAttribute" />
 
     </com.google.android.material.textfield.TextInputLayout>

app/src/main/res/values/themes.xml

@@ -192,6 +192,15 @@
         <item name="android:colorForeground">?attr/settingsSwitchBackgroundColor</item>
     </style>
 
+    <style name="TextInputLayoutTheme" parent="Widget.Design.TextInputLayout">
+        <item name="colorControlActivated">@color/cornflowerBlue</item>
+        <item name="colorControlNormal">@color/cornflowerBlue</item>
+        <item name="colorControlHighlight">@color/cornflowerBlue</item>
+        <item name="android:textColor">@color/grayish</item>
+        <item name="android:textColorSecondary">@color/grayish</item>
+        <item name="android:textColorHint">@color/grayish</item>
+    </style>
+
     <style name="AutoCompleteTextViewStyle" parent="Widget.AppCompat.AutoCompleteTextView">
         <item name="android:textCursorDrawable">@drawable/text_cursor</item>
         <item name="android:textColorHint">?attr/omnibarHintColor</item>