Glossary view is broken

Looks like #1843764: Convert views/templates/views-view-summary-unformatted.tpl.php to twig would also fix it. There is:

+++ /dev/null
@@ -1,20 +0,0 @@
-    <?php if (!empty($row->separator)): ?><? print $row->separator; ?><?php endif; ?>

Do you think we should wait that other issue get in first? Rest of your #1 looks pretty solid.

#9: drupal-1998182-9.patch queued for re-testing.

Looked like #1965048: Frequent random test failure EntityTranslationUITest Invalid values generate a list of form errors..

Tested manually and was about to set this RTBC, but

+++ b/core/modules/views/lib/Drupal/views/Tests/GlossaryTest.php
@@ -55,6 +63,27 @@ public function testGlossaryView() {
+      debug((string) $result[0]);
+      debug(trim(str_replace(array('|', '(', ')'), '', (string) $result[0])));
+      debug($count);

Have to remove these first.

Needs reroll due to twig...

#16: drupal-1998182-16.patch queued for re-testing.

Reroll looks good to me.

Ran Drupal\views\Tests\GlossaryTest locally passes...

Committed 5b5d535 and pushed to 8.x. Thanks!