Issue #3155462 by andrewmacpherson: Remove landmark region role from …

…Powered-by-Drupal block (cherry picked from commit 47a7b73659fddaea8c6fde912999d8cbe456aac0)
drupal · Jul 21, 2020 · 22ceb2d · 22ceb2d
1 parent c0034af
commit 22ceb2d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/modules/system/system.module b/modules/system/system.module
@@ -869,10 +869,6 @@ function system_preprocess_block(&$variables) {
         ];
       }
       break;
-
-    case 'system_powered_by_block':
-      $variables['attributes']['role'] = 'complementary';
-      break;
   }
 }
 

diff --git a/profiles/standard/tests/src/Functional/StandardTest.php b/profiles/standard/tests/src/Functional/StandardTest.php
@@ -61,8 +61,7 @@ public function testStandard() {
     $this->drupalGet('');
     $this->assertText('Main navigation');
 
-    // Verify we have role = aria on system_powered_by and help_block
-    // blocks.
+    // Verify we have role = complementary on help_block blocks.
     $this->drupalGet('admin/structure/block');
     $elements = $this->xpath('//div[@role=:role and @id=:id]', [
       ':role' => 'complementary',
@@ -71,13 +70,6 @@ public function testStandard() {
 
     $this->assertCount(1, $elements, 'Found complementary role on help block.');
 
-    $this->drupalGet('');
-    $elements = $this->xpath('//div[@role=:role and @id=:id]', [
-      ':role' => 'complementary',
-      ':id' => 'block-bartik-powered',
-    ]);
-    $this->assertCount(1, $elements, 'Found complementary role on powered by block.');
-
     // Verify anonymous user can see the block.
     $this->drupalLogout();
     $this->assertText('Main navigation');