From 7dbc725581c5b183b5300fb3f71f01da698518b1 Mon Sep 17 00:00:00 2001
From: Kelly Dwan <ryelle@users.noreply.github.com>
Date: Tue, 12 Nov 2019 12:33:32 -0500
Subject: [PATCH] Style contributors list (first pass)

---
 plugins/wporg-5ftf/assets/css/admin.css       | 21 ++++++++++++++++++
 .../wporg-5ftf/views/manage-contributors.php  | 22 ++++++++-----------
 2 files changed, 30 insertions(+), 13 deletions(-)

diff --git a/plugins/wporg-5ftf/assets/css/admin.css b/plugins/wporg-5ftf/assets/css/admin.css
index a2e35db..a84e352 100755
--- a/plugins/wporg-5ftf/assets/css/admin.css
+++ b/plugins/wporg-5ftf/assets/css/admin.css
@@ -43,3 +43,24 @@
 .pledge-form .email-status.is-unconfirmed {
 	color: #c92c2c;
 }
+
+.contributor-list li * {
+	vertical-align: middle;
+}
+
+.contributor-list .button-link-delete {
+	text-decoration: none;
+	margin-right: 8px;
+}
+
+.contributor-list .avatar {
+	margin-right: 8px;
+}
+
+.contributor-list .contributor-list__name {
+	font-size: 1.1em;
+}
+
+.contributor-list .button {
+	float: right;
+}
diff --git a/plugins/wporg-5ftf/views/manage-contributors.php b/plugins/wporg-5ftf/views/manage-contributors.php
index 54ca076..f8d004b 100644
--- a/plugins/wporg-5ftf/views/manage-contributors.php
+++ b/plugins/wporg-5ftf/views/manage-contributors.php
@@ -28,21 +28,17 @@ namespace WordPressDotOrg\FiveForTheFuture\View;
 						$contributor = get_user_by( 'login', $contributor_post->post_title );
 						?>
 						<li>
-							<?php echo get_avatar( $contributor->user_email, 32 ); ?>
-							<?php echo esc_html( $contributor_post->post_title ); ?>
-							<!-- TODO These buttons don't do anything yet.
-							<button class="button-primary" data-action="remove-contributor" data-contributor-post="<?php echo esc_attr( $contributor_post->ID ); ?>">
-								<?php esc_html_e( 'Remove', 'wporg' ); ?>
+							<button class="button-link button-link-delete" data-action="remove-contributor" data-contributor-post="<?php echo esc_attr( $contributor_post->ID ); ?>" aria-label="<?php esc_html_e( 'Remove', 'wporg' ); ?>">
+								<span class="dashicons dashicons-no-alt" aria-hidden="true"></span>
 							</button>
-							-->
+							<?php echo get_avatar( $contributor->user_email, 32 ); ?>
+							<span class="contributor-list__name">
+								<?php echo esc_html( $contributor_post->post_title ); ?>
+							</span>
 							<?php if ( 'pending' === $contributor_post->post_status ) : ?>
-								<?php submit_button(
-									'Resend Confirmation',
-									'secondary',
-									'resend-contributor-confirmation',
-									false,
-									array( 'formaction' => add_query_arg( 'resend-contributor-id', $contributor_post->ID ) )
-								); ?>
+								<button class="button" data-action="resend-contributor-confirmation" data-contributor-post="<?php echo esc_attr( $contributor_post->ID ); ?>">
+									<?php esc_html_e( 'Resend Confirmation', 'wporg' ); ?>
+								</button>
 							<?php endif; ?>
 						</li>
 					<?php endforeach; ?>