Improve UI for server/index summary pages

1. +++ b/css/search_api.admin.css
   @@ -49,3 +49,14 @@
   + * Index page.
   

   I would call it "Styles for index status page."

2. +++ b/src/Form/IndexStatusForm.php
   @@ -29,6 +29,10 @@ class IndexStatusForm extends FormBase {
   +    // Attach the admin css
   

   Fullstop at the end of sentence is missing.

Rest looks good to me.

Ok fine.

Committed.

Can we add some left margin or padding on the first column?
The word "status" is glued against the border of the box.

And another thing, why didn't we make the background color of the first column match the screenshot in the summary?

@aspilicious: That's fixed by the referenced core issue.

+++ b/css/search_api.admin.css
@@ -49,3 +49,17 @@
+/* Remove animation */
+.search-api-index-status .progress__bar {
+  background-image: none;
+}

Why are we removing the animation from the progress bar on this page? The behaviour of the progress bar should be consistent on every page.

Why are we removing the animation from the progress bar on this page? The behaviour of the progress bar should be consistent on every page.

Since there is no progress currently happening, I don't think the normal progress bar is adequate here. I'm no UI expert, but it just looks very strange. Having a quick overview of the current progress is still useful, though, which is why I think the bar should stay, just unanimated (like it is currently in D7).

Since there is no progress currently happening, I don't think the normal progress bar is adequate here. I'm no UI expert, but it just looks very strange. Having a quick overview of the current progress is still useful, though, which is why I think the bar should stay, just unanimated (like it is currently in D7).

Sure, but changing how the progress bar behaves depending on which module uses it introduces inconsistency for people using these modules. This is a really bad idea. If you have opinions about the current progress bar, that's great, let's talk about that in a core issue. It's possible that we've just identified a situation that we hadn't considered previously or maybe a progress bar is not suitable for the information you're communicating.

I have create #2277551: Add a 'static' variant to the progress bar. I'm not going to create an issue to change the animation direction because we've already justified the current direction and it would imply that I agree. Feel free to create it so we can discuss.

Hi,

I've created a patch for #2277551: Add a 'static' variant to the progress bar and if it is accepted - or whatever has to be done for it to get implemented - you'd need the patch attached here.

Looking at my index's "View" tab, it seems this is solved now?

I'll assume that's as "Yes".