feat: add 5 new WCAG accessibility rules and bump version to 3.0.5

bin/a11y

@@ -19,9 +19,9 @@ use YakNet\AccessibilityConsole\Console\Command\RulesCommand;
 $application = new Application('YakNet Accessibility Console', '3.0.1');
 
 // Add commands
-$application->add(new ScanCommand());
-$application->add(new FixCommand());
-$application->add(new InitCommand());
-$application->add(new RulesCommand());
+$application->addCommand(new ScanCommand());
+$application->addCommand(new FixCommand());
+$application->addCommand(new InitCommand());
+$application->addCommand(new RulesCommand());
 
 $application->run();

composer.json

@@ -12,12 +12,12 @@
         "yaknet"
     ],
     "homepage": "https://forum.yak.net.tr",
-    "version": "3.0.3",
+    "version": "3.0.5",
     "type": "library",
     "license": "MIT",
     "authors": [
         {
-            "name": "YakNet BiliÅŸim",
+            "name": "YakNet Bilişim",
             "email": "[email protected]",
             "role": "Developer"
         }

demo/test_new_error.html

@@ -1,9 +1,25 @@
 <html>
 <head>
     <title>Test Page</title>
+    <meta http-equiv="refresh" content="30">
 </head>
 <body>
-    <img src="avatar.png">
-    <button></button>
+    <img src="avatar.png" alt="Avatar">
+    <button>Submit</button>
+    <a href="#">Click here for nothing</a>
+    <a href="javascript:void(0)">Do something</a>
+    <label for="nonexistent-id">Broken Label</label>
+    <table>
+        <tr>
+            <th>Header 1</th>
+            <th>Header 2</th>
+        </tr>
+        <tr>
+            <td scope="row">Data 1</td>
+            <td>Data 2</td>
+        </tr>
+    </table>
 </body>
 </html>
+
+

src/Rules/LabelTargetExists.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace YakNet\AccessibilityConsole\Rules;
+
+use DOMElement;
+use YakNet\AccessibilityConsole\Core\Severity;
+use YakNet\AccessibilityConsole\Core\Violation;
+use YakNet\AccessibilityConsole\Core\WCAGStandard;
+
+class LabelTargetExists extends AbstractRule
+{
+    public function getId(): string { return 'WCAG_1_3_1_LABEL_TARGET'; }
+    public function getDescription(): string { return 'Label elements with a \'for\' attribute must target an existing element ID.'; }
+    public function getStandard(): WCAGStandard { return WCAGStandard::A; }
+    public function getSeverity(): Severity { return Severity::ERROR; }
+    public function getLevel(): int { return 2; }
+
+    public function check(DOMElement $element): ?Violation
+    {
+        if (strtolower($element->tagName) !== 'label') {
+            return null;
+        }
+
+        if ($element->hasAttribute('for')) {
+            $forId = trim($element->getAttribute('for'));
+            if ($forId === '') {
+                return $this->createViolation(
+                    $element,
+                    $this->getDescription(),
+                    'Provide a valid element ID in the for attribute.'
+                );
+            }
+
+            $doc = $element->ownerDocument;
+            if ($doc !== null) {
+                $xpath = new \DOMXPath($doc);
+                $elements = $xpath->query('//*[@id="' . $forId . '"]');
+                if ($elements === false || $elements->length === 0) {
+                    return $this->createViolation(
+                        $element,
+                        $this->getDescription(),
+                        "Ensure the 'for' attribute value (\"$forId\") matches the ID of a form input element."
+                    );
+                }
+            }
+        }
+
+        return null;
+    }
+}

src/Rules/LinkHrefJavascript.php

@@ -0,0 +1,39 @@
+<?php
+
+namespace YakNet\AccessibilityConsole\Rules;
+
+use DOMElement;
+use YakNet\AccessibilityConsole\Core\Severity;
+use YakNet\AccessibilityConsole\Core\Violation;
+use YakNet\AccessibilityConsole\Core\WCAGStandard;
+
+class LinkHrefJavascript extends AbstractRule
+{
+    public function getId(): string { return 'WCAG_2_1_1_LINK_JAVASCRIPT'; }
+    public function getDescription(): string { return 'Links must not use javascript: void(0) or # as their href. Use buttons for scripting actions.'; }
+    public function getStandard(): WCAGStandard { return WCAGStandard::A; }
+    public function getSeverity(): Severity { return Severity::WARNING; }
+    public function getLevel(): int { return 3; }
+
+    public function check(DOMElement $element): ?Violation
+    {
+        if (strtolower($element->tagName) !== 'a') {
+            return null;
+        }
+
+        if ($element->hasAttribute('href')) {
+            $href = trim($element->getAttribute('href'));
+            
+            // Check for javascript: links or '#' (which represents a placeholder/scripting target)
+            if (str_starts_with(strtolower($href), 'javascript:') || $href === '#') {
+                return $this->createViolation(
+                    $element,
+                    $this->getDescription(),
+                    'Convert the link to a <button> element or provide a valid destination URL.'
+                );
+            }
+        }
+
+        return null;
+    }
+}

src/Rules/MetaRefreshBanned.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace YakNet\AccessibilityConsole\Rules;
+
+use DOMElement;
+use YakNet\AccessibilityConsole\Core\Severity;
+use YakNet\AccessibilityConsole\Core\Violation;
+use YakNet\AccessibilityConsole\Core\WCAGStandard;
+
+class MetaRefreshBanned extends AbstractRule
+{
+    public function getId(): string { return 'WCAG_2_2_1_META_REFRESH'; }
+    public function getDescription(): string { return 'Do not use meta http-equiv="refresh" for automatic page redirects or updates as it disorients users.'; }
+    public function getStandard(): WCAGStandard { return WCAGStandard::A; }
+    public function getSeverity(): Severity { return Severity::ERROR; }
+    public function getLevel(): int { return 1; }
+
+    public function check(DOMElement $element): ?Violation
+    {
+        if (strtolower($element->tagName) !== 'meta') {
+            return null;
+        }
+
+        if ($element->hasAttribute('http-equiv')) {
+            $httpEquiv = trim($element->getAttribute('http-equiv'));
+            if (strtolower($httpEquiv) === 'refresh') {
+                return $this->createViolation(
+                    $element,
+                    $this->getDescription(),
+                    'Use server-side redirects (e.g. HTTP 301/302) or standard JavaScript routing instead of meta refresh.'
+                );
+            }
+        }
+
+        return null;
+    }
+}

src/Rules/ScopeOnlyOnHeaders.php

@@ -0,0 +1,34 @@
+<?php
+
+namespace YakNet\AccessibilityConsole\Rules;
+
+use DOMElement;
+use YakNet\AccessibilityConsole\Core\Severity;
+use YakNet\AccessibilityConsole\Core\Violation;
+use YakNet\AccessibilityConsole\Core\WCAGStandard;
+
+class ScopeOnlyOnHeaders extends AbstractRule
+{
+    public function getId(): string { return 'WCAG_1_3_1_TABLE_SCOPE'; }
+    public function getDescription(): string { return 'The scope attribute is only valid on th elements, not td elements.'; }
+    public function getStandard(): WCAGStandard { return WCAGStandard::A; }
+    public function getSeverity(): Severity { return Severity::ERROR; }
+    public function getLevel(): int { return 2; }
+
+    public function check(DOMElement $element): ?Violation
+    {
+        if (strtolower($element->tagName) !== 'td') {
+            return null;
+        }
+
+        if ($element->hasAttribute('scope')) {
+            return $this->createViolation(
+                $element,
+                $this->getDescription(),
+                'Remove the scope attribute from the td element or change the element to th if it is a header.'
+            );
+        }
+
+        return null;
+    }
+}

src/Rules/TableCaption.php

@@ -0,0 +1,60 @@
+<?php
+
+namespace YakNet\AccessibilityConsole\Rules;
+
+use DOMElement;
+use YakNet\AccessibilityConsole\Core\Severity;
+use YakNet\AccessibilityConsole\Core\Violation;
+use YakNet\AccessibilityConsole\Core\WCAGStandard;
+
+class TableCaption extends AbstractRule
+{
+    public function getId(): string { return 'WCAG_1_3_1_TABLE_CAPTION'; }
+    public function getDescription(): string { return 'Data tables should have a caption element or appropriate ARIA description labels.'; }
+    public function getStandard(): WCAGStandard { return WCAGStandard::A; }
+    public function getSeverity(): Severity { return Severity::WARNING; }
+    public function getLevel(): int { return 3; }
+
+    public function check(DOMElement $element): ?Violation
+    {
+        if (strtolower($element->tagName) !== 'table') {
+            return null;
+        }
+
+        // Data tables usually contain headers (<th> elements). If it is a layout table, this rule is not mandatory.
+        $thElements = $element->getElementsByTagName('th');
+        if ($thElements->length === 0) {
+            return null;
+        }
+
+        // Check if there is a <caption> direct child
+        $hasCaption = false;
+        $captionElements = $element->getElementsByTagName('caption');
+        foreach ($captionElements as $caption) {
+            if ($caption->parentNode === $element && trim($caption->textContent) !== '') {
+                $hasCaption = true;
+                break;
+            }
+        }
+
+        if ($hasCaption) {
+            return null;
+        }
+
+        // Check for aria-label or aria-labelledby or summary attributes
+        $hasAriaLabel = ($element->hasAttribute('aria-label') && trim($element->getAttribute('aria-label')) !== '') ||
+                         ($element->hasAttribute('aria-labelledby') && trim($element->getAttribute('aria-labelledby')) !== '');
+        
+        $hasSummary = $element->hasAttribute('summary') && trim($element->getAttribute('summary')) !== '';
+
+        if ($hasAriaLabel || $hasSummary) {
+            return null;
+        }
+
+        return $this->createViolation(
+            $element,
+            $this->getDescription(),
+            'Add a <caption> element directly inside the <table> or use the aria-label attribute to describe the table.'
+        );
+    }
+}