Read-only hint on index view page

drunken monkey → made their first commit to this issue’s fork.

Thanks a lot for the suggestion, makes sense!
However, it seems Core uses the spelling “read-only” (with hyphen) in the vast majority of cases, so we should go with that, too. I also updated the existing strings (one of them, unfortunately, user-facing).

Then, I think it makes sense to also add a note to the progress bar, and to hide the “Cron batch size” row (since, in fact, nothing will be indexed during cron runs).
I also moved the new row way up, right beneath “Status”.

Finally, you made me realize that the CSS classes we’re using in those tables (ok/warning/error) are actually ignored by at least the Claro default admin theme. I’d guess they were probably used by Seven, which is why we added them, but we failed to notice that Claro changed that. The new classes are the color-* classes. So, I updated also all existing rows that use either the warning or error class to also have the corresponding color-* class. (I kept the existing classes, too, in case someone’s custom admin theme relies on those. Not quite sure about that, though.)
In any case, the index being read-only might be something we want to warn people about so they do notice it, but it’s definitely not an error (generally speaking, that is), so I downgraded the new message accordingly.

Please tell me what you think about these changes!
I realize that some are a bit out of scope, but I think it’s OK to make this a general “lets improve the ’View’ tabs a bit” issue.

Thank you very much for your changes!
For each of your points I give a thumbs-up.
I applied the changes, tested them a bit and they work like a charm.

Great to hear, thanks for reporting back!
Merged. Thanks again!

Automatically closed - issue fixed for 2 weeks with no activity.