Prettify more layouts, fix edit and encrypt intents from key view

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyKeyActivity.java

@@ -193,7 +193,7 @@ public class CertifyKeyActivity extends ActionBarActivity implements
                 if (data.moveToFirst()) {
                     // TODO: put findViewById in onCreate!
                     mPubKeyId = data.getLong(INDEX_MASTER_KEY_ID);
-                    String keyIdStr = PgpKeyHelper.convertKeyIdToHexShort(mPubKeyId);
+                    String keyIdStr = PgpKeyHelper.convertKeyIdToHex(mPubKeyId);
                     ((TextView) findViewById(R.id.key_id)).setText(keyIdStr);
 
                     String mainUserId = data.getString(INDEX_USER_ID);

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/ViewKeyMainFragment.java

@@ -39,6 +39,7 @@ import com.beardedhen.androidbootstrap.BootstrapButton;
 import org.sufficientlysecure.keychain.Constants;
 import org.sufficientlysecure.keychain.R;
 import org.sufficientlysecure.keychain.pgp.PgpKeyHelper;
+import org.sufficientlysecure.keychain.provider.KeychainContract.KeyRingData;
 import org.sufficientlysecure.keychain.provider.KeychainContract.KeyRings;
 import org.sufficientlysecure.keychain.provider.KeychainContract.Keys;
 import org.sufficientlysecure.keychain.provider.KeychainContract.UserIds;
@@ -204,8 +205,9 @@ public class ViewKeyMainFragment extends Fragment implements
          * because the notification triggers faster than the activity closes.
          */
         // Avoid NullPointerExceptions...
-        if(data.getCount() == 0)
+        if(data.getCount() == 0) {
             return;
+        }
         // Swap the new cursor in. (The framework will take care of closing the
         // old cursor once we return.)
         switch (loader.getId()) {
@@ -232,7 +234,8 @@ public class ViewKeyMainFragment extends Fragment implements
                         mActionEdit.setOnClickListener(new View.OnClickListener() {
                             public void onClick(View view) {
                                 Intent editIntent = new Intent(getActivity(), EditKeyActivity.class);
-                                editIntent.setData(mDataUri);
+                                editIntent.setData(
+                                        KeyRingData.buildSecretKeyRingUri(mDataUri));
                                 editIntent.setAction(EditKeyActivity.ACTION_EDIT_KEY);
                                 startActivityForResult(editIntent, 0);
                             }
@@ -330,7 +333,8 @@ public class ViewKeyMainFragment extends Fragment implements
     private void encryptToContact(Uri dataUri) {
         // TODO preselect from uri? should be feasible without trivial query
         try {
-            long keyId = ProviderHelper.getMasterKeyId(getActivity(), dataUri);
+            long keyId = ProviderHelper.getMasterKeyId(getActivity(),
+                    KeyRingData.buildPublicKeyRingUri(dataUri));
 
             long[] encryptionKeyIds = new long[]{ keyId };
             Intent intent = new Intent(getActivity(), EncryptActivity.class);

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/adapter/ViewKeyUserIdsAdapter.java

@@ -28,7 +28,9 @@ import android.widget.CheckBox;
 import android.widget.CompoundButton;
 import android.widget.ImageView;
 import android.widget.TextView;
+
 import org.sufficientlysecure.keychain.R;
+import org.sufficientlysecure.keychain.helper.OtherHelper;
 import org.sufficientlysecure.keychain.pgp.PgpKeyHelper;
 import org.sufficientlysecure.keychain.provider.KeychainContract.UserIds;
 import org.sufficientlysecure.keychain.provider.KeychainContract.Certs;
@@ -57,6 +59,7 @@ public class ViewKeyUserIdsAdapter extends CursorAdapter implements AdapterView.
 
         initIndex(c);
     }
+
     public ViewKeyUserIdsAdapter(Context context, Cursor c, int flags) {
         this(context, c, flags, false);
     }
@@ -122,16 +125,26 @@ public class ViewKeyUserIdsAdapter extends CursorAdapter implements AdapterView.
         if (cursor.getInt(mIsRevoked) > 0) {
             vRank.setText(" ");
             vVerified.setImageResource(android.R.drawable.presence_away);
+
+            // disable and strike through text for revoked user ids
+            vUserId.setEnabled(false);
+            vAddress.setEnabled(false);
+            vUserId.setText(OtherHelper.strikeOutText(vUserId.getText()));
+            vAddress.setText(OtherHelper.strikeOutText(vAddress.getText()));
         } else {
+            vUserId.setEnabled(true);
+            vAddress.setEnabled(true);
+
             int verified = cursor.getInt(mVerifiedId);
             // TODO introduce own resources for this :)
-            if(verified == Certs.VERIFIED_SECRET)
+            if (verified == Certs.VERIFIED_SECRET) {
                 vVerified.setImageResource(android.R.drawable.presence_online);
-            else if(verified == Certs.VERIFIED_SELF)
+            } else if (verified == Certs.VERIFIED_SELF) {
                 vVerified.setImageResource(android.R.drawable.presence_invisible);
-            else
+            } else {
                 vVerified.setImageResource(android.R.drawable.presence_busy);
             }
+        }
 
         // don't care further if checkboxes aren't shown
         if (mCheckStates == null) {

OpenKeychain/src/main/res/layout/certify_key_activity.xml

@@ -77,8 +77,7 @@
                 <TextView
                     android:id="@+id/main_user_id"
                     android:layout_width="match_parent"
-                    android:layout_height="wrap_content"
+                    android:layout_height="wrap_content" />
-                    android:typeface="monospace" />
 
             </TableRow>
 

OpenKeychain/src/main/res/layout/select_secret_key_layout_fragment.xml

@@ -52,8 +52,7 @@
             android:layout_marginRight="5dip"
             android:text=""
             android:visibility="gone"
-            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textAppearance="?android:attr/textAppearanceSmall" />
-            android:paddingLeft="10dp" />
 
         <TextView
             android:id="@+id/select_secret_key_master_key_hex"

OpenKeychain/src/main/res/layout/view_cert_activity.xml

@@ -61,7 +61,8 @@
                     android:id="@+id/signee_key"
                     android:layout_width="0dp"
                     android:layout_height="wrap_content"
-                    android:paddingRight="5dip" />
+                    android:paddingRight="5dip"
+                    android:typeface="monospace" />
             </TableRow>
 
             <TableRow>