Icon or text - #3424744
1 unresolved thread
1 unresolved thread
Closes #3424744
Merge request reports
Activity
119 // phpcs:disable 120 $links = [ 121 1 => MenuLinkMock::create(['id' => 'test.example1', 'route_name' => 'example1', 'title' => 'title 1', 'parent' => '', 'weight' => 0]), 122 2 => MenuLinkMock::create(['id' => 'test.example2', 'route_name' => 'example2', 'title' => 'Another title', 'parent' => '', 'route_parameters' => ['foo' => 'bar'], 'weight' => 1]), 123 3 => MenuLinkMock::create(['id' => 'test.example3', 'route_name' => 'example3', 'title' => 'A menu link', 'parent' => 'test.example2', 'weight' => 2]), 124 ]; 125 // phpcs:enable 126 foreach ($links as $instance) { 127 $this->menuLinkManager->addDefinition($instance->getPluginId(), $instance->getPluginDefinition()); 128 } 129 } 130 131 /** 132 * Tests the generated HTML markup. 133 */ 134 public function testToolbarButtonAttributes() { changed this line in version 3 of the diff
- Resolved by Cristina Chumillas
- Resolved by Cristina Chumillas
2 small design fixes:
- On a design perspective the letters need to be centered, both vertically and horizontally.
- For legibility purposes, this should use the first letter as capitalized, but not the second. The designs already have Caps + lower case. The reason is that lower case's shapes are way better to recognize.
- Resolved by Cristina Chumillas
- Resolved by Cristina Chumillas
2 small design fixes:
- On a design perspective the letters need to be centered, both vertically and horizontally.
- For legibility purposes, this should use the first letter as capitalized, but not the second. The designs already have Caps + lower case. The reason is that lower case's shapes are way better to recognize.
Please register or sign in to reply