From aa6d0dd86773eda7d0a5a6855b40c2e1f95e2b45 Mon Sep 17 00:00:00 2001 From: Vincent Breitmoser Date: Thu, 26 Mar 2015 13:53:43 +0100 Subject: some linked id ui fine tuning --- .../keychain/pgp/linked/resources/DnsResource.java | 2 +- .../pgp/linked/resources/GenericHttpsResource.java | 2 +- .../pgp/linked/resources/GithubResource.java | 2 +- .../pgp/linked/resources/TwitterResource.java | 2 +- .../keychain/ui/linked/LinkedIdViewFragment.java | 53 ++++++++++++--------- OpenKeychain/src/main/res/anim/fade_in_down.xml | 11 +++++ OpenKeychain/src/main/res/anim/fade_in_quick.xml | 7 +++ OpenKeychain/src/main/res/anim/fade_out_quick.xml | 7 +++ OpenKeychain/src/main/res/anim/fade_out_up.xml | 11 +++++ .../src/main/res/drawable-hdpi/linked_dns.png | Bin 0 -> 8141 bytes .../src/main/res/drawable-hdpi/linked_github.png | Bin 0 -> 1464 bytes .../src/main/res/drawable-hdpi/linked_https.png | Bin 0 -> 2526 bytes .../src/main/res/drawable-hdpi/linked_twitter.png | Bin 0 -> 1785 bytes .../src/main/res/drawable-mdpi/linked_dns.png | Bin 0 -> 5238 bytes .../src/main/res/drawable-mdpi/linked_github.png | Bin 0 -> 1107 bytes .../src/main/res/drawable-mdpi/linked_https.png | Bin 0 -> 1836 bytes .../src/main/res/drawable-mdpi/linked_twitter.png | Bin 0 -> 1353 bytes .../src/main/res/drawable-xhdpi/linked_dns.png | Bin 0 -> 14798 bytes .../src/main/res/drawable-xhdpi/linked_github.png | Bin 0 -> 2376 bytes .../src/main/res/drawable-xhdpi/linked_https.png | Bin 0 -> 3620 bytes .../src/main/res/drawable-xhdpi/linked_twitter.png | Bin 0 -> 2125 bytes .../src/main/res/drawable-xxhdpi/linked_dns.png | Bin 0 -> 22410 bytes .../src/main/res/drawable-xxhdpi/linked_github.png | Bin 0 -> 3404 bytes .../src/main/res/drawable-xxhdpi/linked_https.png | Bin 0 -> 4811 bytes .../main/res/drawable-xxhdpi/linked_twitter.png | Bin 0 -> 2840 bytes OpenKeychain/src/main/res/drawable/dns.png | Bin 2043 -> 0 bytes OpenKeychain/src/main/res/drawable/github.png | Bin 1714 -> 0 bytes OpenKeychain/src/main/res/drawable/ssl_lock.png | Bin 479 -> 0 bytes OpenKeychain/src/main/res/drawable/twitter.png | Bin 5122 -> 0 bytes .../src/main/res/layout/cert_list_widget.xml | 1 - .../layout/linked_create_dns_fragment_step1.xml | 28 +++-------- .../layout/linked_create_github_fragment_step1.xml | 31 ++++-------- .../layout/linked_create_https_fragment_step1.xml | 29 +++-------- .../linked_create_twitter_fragment_step1.xml | 30 +++--------- .../src/main/res/layout/linked_create_verify.xml | 4 +- .../src/main/res/layout/linked_id_item.xml | 12 +++-- .../main/res/layout/linked_id_view_fragment.xml | 36 +++++++------- .../src/main/res/layout/linked_select_fragment.xml | 24 +++++----- OpenKeychain/src/main/res/values/strings.xml | 10 +++- 39 files changed, 149 insertions(+), 153 deletions(-) create mode 100644 OpenKeychain/src/main/res/anim/fade_in_down.xml create mode 100644 OpenKeychain/src/main/res/anim/fade_in_quick.xml create mode 100644 OpenKeychain/src/main/res/anim/fade_out_quick.xml create mode 100644 OpenKeychain/src/main/res/anim/fade_out_up.xml create mode 100644 OpenKeychain/src/main/res/drawable-hdpi/linked_dns.png create mode 100644 OpenKeychain/src/main/res/drawable-hdpi/linked_github.png create mode 100644 OpenKeychain/src/main/res/drawable-hdpi/linked_https.png create mode 100644 OpenKeychain/src/main/res/drawable-hdpi/linked_twitter.png create mode 100644 OpenKeychain/src/main/res/drawable-mdpi/linked_dns.png create mode 100644 OpenKeychain/src/main/res/drawable-mdpi/linked_github.png create mode 100644 OpenKeychain/src/main/res/drawable-mdpi/linked_https.png create mode 100644 OpenKeychain/src/main/res/drawable-mdpi/linked_twitter.png create mode 100644 OpenKeychain/src/main/res/drawable-xhdpi/linked_dns.png create mode 100644 OpenKeychain/src/main/res/drawable-xhdpi/linked_github.png create mode 100644 OpenKeychain/src/main/res/drawable-xhdpi/linked_https.png create mode 100644 OpenKeychain/src/main/res/drawable-xhdpi/linked_twitter.png create mode 100644 OpenKeychain/src/main/res/drawable-xxhdpi/linked_dns.png create mode 100644 OpenKeychain/src/main/res/drawable-xxhdpi/linked_github.png create mode 100644 OpenKeychain/src/main/res/drawable-xxhdpi/linked_https.png create mode 100644 OpenKeychain/src/main/res/drawable-xxhdpi/linked_twitter.png delete mode 100644 OpenKeychain/src/main/res/drawable/dns.png delete mode 100644 OpenKeychain/src/main/res/drawable/github.png delete mode 100644 OpenKeychain/src/main/res/drawable/ssl_lock.png delete mode 100644 OpenKeychain/src/main/res/drawable/twitter.png (limited to 'OpenKeychain/src/main') diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/DnsResource.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/DnsResource.java index 368d0f4da..a8faa435d 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/DnsResource.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/DnsResource.java @@ -112,7 +112,7 @@ public class DnsResource extends LinkedCookieResource { @Override public @DrawableRes int getDisplayIcon() { - return R.drawable.dns; + return R.drawable.linked_dns; } @Override diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GenericHttpsResource.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GenericHttpsResource.java index 597d5aee0..8f5c0f8c2 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GenericHttpsResource.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GenericHttpsResource.java @@ -65,7 +65,7 @@ public class GenericHttpsResource extends LinkedCookieResource { @Override public @DrawableRes int getDisplayIcon() { - return R.drawable.ssl_lock; + return R.drawable.linked_https; } @Override diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GithubResource.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GithubResource.java index 723898d20..d411395a3 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GithubResource.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/GithubResource.java @@ -182,7 +182,7 @@ public class GithubResource extends LinkedCookieResource { @Override public @DrawableRes int getDisplayIcon() { - return R.drawable.github; + return R.drawable.linked_github; } @Override diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/TwitterResource.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/TwitterResource.java index 54444ee5e..935268da6 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/TwitterResource.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/linked/resources/TwitterResource.java @@ -108,7 +108,7 @@ public class TwitterResource extends LinkedCookieResource { @Override public @DrawableRes int getDisplayIcon() { - return R.drawable.twitter; + return R.drawable.linked_twitter; } @Override diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/linked/LinkedIdViewFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/linked/LinkedIdViewFragment.java index e0db0174e..aa99a2907 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/linked/LinkedIdViewFragment.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/linked/LinkedIdViewFragment.java @@ -25,10 +25,11 @@ import android.view.View; import android.view.View.OnClickListener; import android.view.ViewGroup; import android.widget.ImageView; -import android.widget.TextView; +import android.widget.TextSwitcher; import android.widget.ViewAnimator; import org.sufficientlysecure.keychain.Constants; +import org.sufficientlysecure.keychain.Constants.key; import org.sufficientlysecure.keychain.R; import org.sufficientlysecure.keychain.operations.results.CertifyResult; import org.sufficientlysecure.keychain.operations.results.LinkedVerifyResult; @@ -180,16 +181,12 @@ public class LinkedIdViewFragment extends Fragment implements private void loadIdentity(RawLinkedIdentity linkedId, int certStatus) { mLinkedId = linkedId; - setShowVerifying(false); - if (mLinkedId instanceof LinkedIdentity) { LinkedResource res = ((LinkedIdentity) mLinkedId).mResource; mLinkedResource = (LinkedCookieResource) res; } if (!mIsSecret) { - mViewHolder.mLinkedIdHolder.vVerified.setVisibility(View.VISIBLE); - switch (certStatus) { case Certs.VERIFIED_SECRET: KeyFormattingUtils.setStatusImage(mContext, mViewHolder.mLinkedIdHolder.vVerified, @@ -204,12 +201,12 @@ public class LinkedIdViewFragment extends Fragment implements null, State.INVALID, KeyFormattingUtils.DEFAULT_COLOR); break; } - } else { - mViewHolder.mLinkedIdHolder.vVerified.setVisibility(View.GONE); } mViewHolder.mLinkedIdHolder.setData(mContext, mLinkedId); + setShowVerifying(false); + // no resource, nothing further we can do… if (mLinkedResource == null) { mViewHolder.vButtonView.setVisibility(View.GONE); @@ -255,7 +252,7 @@ public class LinkedIdViewFragment extends Fragment implements private final View vButtonBack; private final ViewAnimator vProgress; - private final TextView vText; + private final TextSwitcher vText; ViewHolder(View root) { vLinkedCerts = (CertListWidget) root.findViewById(R.id.linked_id_certs); @@ -274,18 +271,18 @@ public class LinkedIdViewFragment extends Fragment implements vItemCertified = (ViewAnimator) root.findViewById(R.id.linked_id_certified); vProgress = (ViewAnimator) root.findViewById(R.id.linked_cert_progress); - vText = (TextView) root.findViewById(R.id.linked_cert_text); + vText = (TextSwitcher) root.findViewById(R.id.linked_cert_text); } enum VerifyState { VERIFYING, VERIFY_OK, VERIFY_ERROR, CERTIFYING } - void setVerifyingState(VerifyState state, boolean isSecret) { + void setVerifyingState(Context context, VerifyState state, boolean isSecret) { switch (state) { case VERIFYING: vProgress.setDisplayedChild(0); - vText.setText("Verifying…"); + vText.setText(context.getString(R.string.linked_text_verifying)); vKeySpinner.setVisibility(View.GONE); break; @@ -303,25 +300,30 @@ public class LinkedIdViewFragment extends Fragment implements case VERIFY_ERROR: showButton(1); vProgress.setDisplayedChild(2); - vText.setText("Error"); + vText.setText(context.getString(R.string.linked_text_error)); vKeySpinner.setVisibility(View.GONE); break; case CERTIFYING: vProgress.setDisplayedChild(0); - vText.setText("Confirming…"); + vText.setText(context.getString(R.string.linked_text_confirming)); vKeySpinner.setVisibility(View.GONE); break; } } - void showVerifyingContainer(boolean show, boolean isSecret) { + void showVerifyingContainer(Context context, boolean show, boolean isSecret) { if (vVerifyingContainer.getDisplayedChild() == (show ? 1 : 0)) { return; } + vVerifyingContainer.setInAnimation(context, show ? R.anim.fade_in_up : R.anim.fade_in_down); + vVerifyingContainer.setOutAnimation(context, show ? R.anim.fade_out_up : R.anim.fade_out_down); vVerifyingContainer.setDisplayedChild(show ? 1 : 0); - vItemCertified.setDisplayedChild(show && !isSecret ? 1 : 0); + + vItemCertified.setInAnimation(context, show ? R.anim.fade_in_up : R.anim.fade_in_down); + vItemCertified.setOutAnimation(context, show ? R.anim.fade_out_up : R.anim.fade_out_down); + vItemCertified.setDisplayedChild(show || isSecret ? 1 : 0); } void showButton(int which) { @@ -363,7 +365,7 @@ public class LinkedIdViewFragment extends Fragment implements mViewHolder.showButton(0); mViewHolder.vKeySpinner.setVisibility(View.GONE); - mViewHolder.showVerifyingContainer(false, mIsSecret); + mViewHolder.showVerifyingContainer(mContext, false, mIsSecret); return; } @@ -376,7 +378,7 @@ public class LinkedIdViewFragment extends Fragment implements manager.beginTransaction().addToBackStack("verification").commit(); manager.executePendingTransactions(); manager.addOnBackStackChangedListener(this); - mViewHolder.showVerifyingContainer(true, mIsSecret); + mViewHolder.showVerifyingContainer(mContext, true, mIsSecret); } @@ -449,7 +451,7 @@ public class LinkedIdViewFragment extends Fragment implements setShowVerifying(true); mViewHolder.vKeySpinner.setVisibility(View.GONE); - mViewHolder.setVerifyingState(VerifyState.VERIFYING, mIsSecret); + mViewHolder.setVerifyingState(mContext, VerifyState.VERIFYING, mIsSecret); mInProgress = new AsyncTask() { @Override @@ -474,10 +476,10 @@ public class LinkedIdViewFragment extends Fragment implements return; } if (result.success()) { - mViewHolder.vText.setText(mLinkedResource.getVerifiedText(mIsSecret)); - mViewHolder.setVerifyingState(VerifyState.VERIFY_OK, mIsSecret); + mViewHolder.vText.setText(getString(mLinkedResource.getVerifiedText(mIsSecret))); + mViewHolder.setVerifyingState(mContext, VerifyState.VERIFY_OK, mIsSecret); } else { - mViewHolder.setVerifyingState(VerifyState.VERIFY_ERROR, mIsSecret); + mViewHolder.setVerifyingState(mContext, VerifyState.VERIFY_ERROR, mIsSecret); result.createNotify(getActivity()).show(); } mInProgress = null; @@ -493,8 +495,12 @@ public class LinkedIdViewFragment extends Fragment implements } // get the user's passphrase for this key (if required) - Passphrase passphrase; long certifyKeyId = mViewHolder.vKeySpinner.getSelectedItemId(); + if (certifyKeyId == key.none || certifyKeyId == key.symmetric) { + Notify.create(getActivity(), R.string.select_key_to_certify, Style.ERROR).show(); + } + + Passphrase passphrase; try { passphrase = PassphraseCacheService.getCachedPassphrase( getActivity(), certifyKeyId, certifyKeyId); @@ -541,7 +547,7 @@ public class LinkedIdViewFragment extends Fragment implements return; } - mViewHolder.setVerifyingState(VerifyState.CERTIFYING, false); + mViewHolder.setVerifyingState(mContext, VerifyState.CERTIFYING, false); Bundle data = new Bundle(); { @@ -577,6 +583,7 @@ public class LinkedIdViewFragment extends Fragment implements if (message.arg1 == MessageStatus.OKAY.ordinal()) { CertifyResult result = data.getParcelable(CertifyResult.EXTRA_RESULT); result.createNotify(getActivity()).show(); + // no need to do anything else, we will get a loader refresh! } } diff --git a/OpenKeychain/src/main/res/anim/fade_in_down.xml b/OpenKeychain/src/main/res/anim/fade_in_down.xml new file mode 100644 index 000000000..fb9ed416e --- /dev/null +++ b/OpenKeychain/src/main/res/anim/fade_in_down.xml @@ -0,0 +1,11 @@ + + + + + \ No newline at end of file diff --git a/OpenKeychain/src/main/res/anim/fade_in_quick.xml b/OpenKeychain/src/main/res/anim/fade_in_quick.xml new file mode 100644 index 000000000..e0725de3d --- /dev/null +++ b/OpenKeychain/src/main/res/anim/fade_in_quick.xml @@ -0,0 +1,7 @@ + + + + \ No newline at end of file diff --git a/OpenKeychain/src/main/res/anim/fade_out_quick.xml b/OpenKeychain/src/main/res/anim/fade_out_quick.xml new file mode 100644 index 000000000..94fc508d7 --- /dev/null +++ b/OpenKeychain/src/main/res/anim/fade_out_quick.xml @@ -0,0 +1,7 @@ + + + + \ No newline at end of file diff --git a/OpenKeychain/src/main/res/anim/fade_out_up.xml b/OpenKeychain/src/main/res/anim/fade_out_up.xml new file mode 100644 index 000000000..65049a387 --- /dev/null +++ b/OpenKeychain/src/main/res/anim/fade_out_up.xml @@ -0,0 +1,11 @@ + + + + + \ No newline at end of file diff --git a/OpenKeychain/src/main/res/drawable-hdpi/linked_dns.png b/OpenKeychain/src/main/res/drawable-hdpi/linked_dns.png new file mode 100644 index 000000000..898160fd1 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-hdpi/linked_dns.png differ diff --git a/OpenKeychain/src/main/res/drawable-hdpi/linked_github.png b/OpenKeychain/src/main/res/drawable-hdpi/linked_github.png new file mode 100644 index 000000000..bba0729ff Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-hdpi/linked_github.png differ diff --git a/OpenKeychain/src/main/res/drawable-hdpi/linked_https.png b/OpenKeychain/src/main/res/drawable-hdpi/linked_https.png new file mode 100644 index 000000000..83585a890 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-hdpi/linked_https.png differ diff --git a/OpenKeychain/src/main/res/drawable-hdpi/linked_twitter.png b/OpenKeychain/src/main/res/drawable-hdpi/linked_twitter.png new file mode 100644 index 000000000..f80f9cc07 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-hdpi/linked_twitter.png differ diff --git a/OpenKeychain/src/main/res/drawable-mdpi/linked_dns.png b/OpenKeychain/src/main/res/drawable-mdpi/linked_dns.png new file mode 100644 index 000000000..6dbbf6fd6 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-mdpi/linked_dns.png differ diff --git a/OpenKeychain/src/main/res/drawable-mdpi/linked_github.png b/OpenKeychain/src/main/res/drawable-mdpi/linked_github.png new file mode 100644 index 000000000..a2dd32029 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-mdpi/linked_github.png differ diff --git a/OpenKeychain/src/main/res/drawable-mdpi/linked_https.png b/OpenKeychain/src/main/res/drawable-mdpi/linked_https.png new file mode 100644 index 000000000..6ffd0bef7 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-mdpi/linked_https.png differ diff --git a/OpenKeychain/src/main/res/drawable-mdpi/linked_twitter.png b/OpenKeychain/src/main/res/drawable-mdpi/linked_twitter.png new file mode 100644 index 000000000..5b0ca3f40 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-mdpi/linked_twitter.png differ diff --git a/OpenKeychain/src/main/res/drawable-xhdpi/linked_dns.png b/OpenKeychain/src/main/res/drawable-xhdpi/linked_dns.png new file mode 100644 index 000000000..5056cb0b1 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xhdpi/linked_dns.png differ diff --git a/OpenKeychain/src/main/res/drawable-xhdpi/linked_github.png b/OpenKeychain/src/main/res/drawable-xhdpi/linked_github.png new file mode 100644 index 000000000..b5a861ad1 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xhdpi/linked_github.png differ diff --git a/OpenKeychain/src/main/res/drawable-xhdpi/linked_https.png b/OpenKeychain/src/main/res/drawable-xhdpi/linked_https.png new file mode 100644 index 000000000..9973bfde2 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xhdpi/linked_https.png differ diff --git a/OpenKeychain/src/main/res/drawable-xhdpi/linked_twitter.png b/OpenKeychain/src/main/res/drawable-xhdpi/linked_twitter.png new file mode 100644 index 000000000..35e22cafe Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xhdpi/linked_twitter.png differ diff --git a/OpenKeychain/src/main/res/drawable-xxhdpi/linked_dns.png b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_dns.png new file mode 100644 index 000000000..f28ebc695 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_dns.png differ diff --git a/OpenKeychain/src/main/res/drawable-xxhdpi/linked_github.png b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_github.png new file mode 100644 index 000000000..38d7a12f4 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_github.png differ diff --git a/OpenKeychain/src/main/res/drawable-xxhdpi/linked_https.png b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_https.png new file mode 100644 index 000000000..25e1baf28 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_https.png differ diff --git a/OpenKeychain/src/main/res/drawable-xxhdpi/linked_twitter.png b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_twitter.png new file mode 100644 index 000000000..62116fd34 Binary files /dev/null and b/OpenKeychain/src/main/res/drawable-xxhdpi/linked_twitter.png differ diff --git a/OpenKeychain/src/main/res/drawable/dns.png b/OpenKeychain/src/main/res/drawable/dns.png deleted file mode 100644 index 69d0a4fa8..000000000 Binary files a/OpenKeychain/src/main/res/drawable/dns.png and /dev/null differ diff --git a/OpenKeychain/src/main/res/drawable/github.png b/OpenKeychain/src/main/res/drawable/github.png deleted file mode 100644 index 8b25551a9..000000000 Binary files a/OpenKeychain/src/main/res/drawable/github.png and /dev/null differ diff --git a/OpenKeychain/src/main/res/drawable/ssl_lock.png b/OpenKeychain/src/main/res/drawable/ssl_lock.png deleted file mode 100644 index 00c4d8e4f..000000000 Binary files a/OpenKeychain/src/main/res/drawable/ssl_lock.png and /dev/null differ diff --git a/OpenKeychain/src/main/res/drawable/twitter.png b/OpenKeychain/src/main/res/drawable/twitter.png deleted file mode 100644 index 3533e0488..000000000 Binary files a/OpenKeychain/src/main/res/drawable/twitter.png and /dev/null differ diff --git a/OpenKeychain/src/main/res/layout/cert_list_widget.xml b/OpenKeychain/src/main/res/layout/cert_list_widget.xml index 715e4bfa5..7052a1901 100644 --- a/OpenKeychain/src/main/res/layout/cert_list_widget.xml +++ b/OpenKeychain/src/main/res/layout/cert_list_widget.xml @@ -12,7 +12,6 @@ android:layout_width="fill_parent" android:layout_height="wrap_content" android:orientation="horizontal" - android:minHeight="?android:attr/listPreferredItemHeight" > - - - - - - - + android:textAppearance="?android:attr/textAppearanceMedium" + android:text="@string/linked_create_dns_1_1" /> - - - - - - - + android:layout_gravity="center_vertical" + android:drawableLeft="@drawable/linked_github" + android:drawablePadding="8dp" + android:textAppearance="?android:attr/textAppearanceMedium" + android:text="@string/linked_create_github_1_1" /> - - - - - - - + android:drawableLeft="@drawable/linked_https" + android:drawablePadding="8dp" + android:textAppearance="?android:attr/textAppearanceMedium" + android:text="@string/linked_create_https_1_1" /> - - - - - - - + android:drawableLeft="@drawable/linked_twitter" + android:drawablePadding="8dp" + android:textAppearance="?android:attr/textAppearanceMedium" + android:text="@string/linked_create_twitter_1_1" /> @@ -66,7 +66,7 @@ android:layout_width="wrap_content" android:layout_height="wrap_content" style="?android:buttonBarButtonStyle" - android:text="Retry" + android:text="@string/linked_button_retry" android:id="@+id/button_retry" /> diff --git a/OpenKeychain/src/main/res/layout/linked_id_item.xml b/OpenKeychain/src/main/res/layout/linked_id_item.xml index 900d3a5d5..6b39144a8 100644 --- a/OpenKeychain/src/main/res/layout/linked_id_item.xml +++ b/OpenKeychain/src/main/res/layout/linked_id_item.xml @@ -14,7 +14,7 @@ android:id="@+id/linked_id_type_icon" android:layout_marginLeft="14dp" android:layout_marginStart="14dp" - android:src="@drawable/dns" + tools:src="@drawable/linked_dns" android:layout_gravity="center_vertical" android:scaleType="fitCenter" /> @@ -32,16 +32,18 @@ android:id="@+id/linked_id_title" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:text="Title" - android:textAppearance="?android:attr/textAppearanceMedium" /> + tools:text="Title" + android:textAppearance="?android:attr/textAppearanceMedium" + /> + tools:text="comment" + android:textAppearance="?android:attr/textAppearanceSmall" + /> diff --git a/OpenKeychain/src/main/res/layout/linked_id_view_fragment.xml b/OpenKeychain/src/main/res/layout/linked_id_view_fragment.xml index 2fb79a4bf..e25dfca70 100644 --- a/OpenKeychain/src/main/res/layout/linked_id_view_fragment.xml +++ b/OpenKeychain/src/main/res/layout/linked_id_view_fragment.xml @@ -28,7 +28,7 @@ style="@style/CardViewHeader" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:text="Linked Identity" /> + android:text="@string/card_linked_identity" /> @@ -36,10 +36,9 @@ android:layout_width="fill_parent" android:layout_height="wrap_content" android:id="@+id/linked_verify_container" - android:inAnimation="@anim/fade_in" - android:outAnimation="@anim/fade_out" android:layout_marginLeft="12dp" android:layout_marginRight="12dp" + android:measureAllChildren="false" > @@ -47,28 +46,33 @@ + android:baselineAligned="false" + android:animateLayoutChanges="true" + > - - + + + @@ -152,21 +156,21 @@ android:id="@+id/button_verify" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:text="Verify" + android:text="@string/linked_button_verify" android:textColor="@color/link_text_material_light" style="?android:attr/borderlessButtonStyle" />