From b6124d7371baec960776356bf8ee2cd7d8b00801 Mon Sep 17 00:00:00 2001
From: Marius Hein <marius.hein@netways.de>
Date: Mon, 23 Feb 2015 11:18:58 +0100
Subject: [PATCH] Header: Remove skiplinks

Skiplinks are to much and confusing for other users. Better is to use
header navigation in screen readers.

refs #7076
---
 .../config/authentication/reorder.phtml       | 10 -----
 .../views/scripts/config/resource.phtml       | 10 -----
 application/views/scripts/roles/index.phtml   | 10 -----
 .../views/scripts/alertsummary/index.phtml    | 30 --------------
 .../views/scripts/process/info.phtml          | 41 +------------------
 5 files changed, 1 insertion(+), 100 deletions(-)

diff --git a/application/views/scripts/config/authentication/reorder.phtml b/application/views/scripts/config/authentication/reorder.phtml
index f26eba82e..4583a23b3 100644
--- a/application/views/scripts/config/authentication/reorder.phtml
+++ b/application/views/scripts/config/authentication/reorder.phtml
@@ -5,16 +5,6 @@
   <h1 tabindex="-1" id="authentication-reorder">
     <?= $this->translate('Authentication Configuration'); ?>
   </h1>
-  <div class="skip-links skip-links-inline">
-    <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-    <ul>
-      <li>
-        <a tabindex="0" href="#authentication-reorder-form">
-          <?= t('Skip To Authentication Reorder Form'); ?>
-        </a>
-      </li>
-    </ul>
-  </div>
   <h2 tabindex="-1" id="authentication-new-backend" class="sr-only">
     <?= t('New Authentication Backend'); ?>
   </h2>
diff --git a/application/views/scripts/config/resource.phtml b/application/views/scripts/config/resource.phtml
index 6964c48cc..c4eb9360f 100644
--- a/application/views/scripts/config/resource.phtml
+++ b/application/views/scripts/config/resource.phtml
@@ -5,16 +5,6 @@
   <h1 tabindex="-1" id="resource-index">
     <?= t('Resource Configuration'); ?>
   </h1>
-  <div class="skip-links skip-links-inline">
-    <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-    <ul>
-      <li>
-        <a tabindex="0" href="#resource-edit-table">
-          <?= t('Skip To Existing Resources'); ?>
-        </a>
-      </li>
-    </ul>
-  </div>
   <h2 tabindex="-1" id="resource-new-resource" class="sr-only">
     <?= t('Create New Resource'); ?>
   </h2>
diff --git a/application/views/scripts/roles/index.phtml b/application/views/scripts/roles/index.phtml
index 61cbe3310..d4f5ad2aa 100644
--- a/application/views/scripts/roles/index.phtml
+++ b/application/views/scripts/roles/index.phtml
@@ -5,16 +5,6 @@
     <h1 tabindex="-1" id="roles-index">
         <?= $this->translate('Roles') ?>
     </h1>
-    <div class="skip-links skip-links-inline">
-        <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-        <ul>
-            <li>
-                <a tabindex="0" href="#roles-index-new">
-                    <?= t('Skip To Create New Role'); ?>
-                </a>
-            </li>
-        </ul>
-    </div>
     <div>
         <?php /** @var \Icinga\Application\Config $roles */ if ($roles->isEmpty()): ?>
             <?= $this->translate('No roles found.') ?>
diff --git a/modules/monitoring/application/views/scripts/alertsummary/index.phtml b/modules/monitoring/application/views/scripts/alertsummary/index.phtml
index 7db60791a..d3e7028c0 100644
--- a/modules/monitoring/application/views/scripts/alertsummary/index.phtml
+++ b/modules/monitoring/application/views/scripts/alertsummary/index.phtml
@@ -11,36 +11,6 @@
     <h1 tabindex="-1">
         <?= $this->translate('Alert summary'); ?>
     </h1>
-    <div class="skip-links skip-links-inline">
-        <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-        <ul>
-            <li>
-                <a tabindex="0" href="#alertsummary-notifications">
-                    <?= t('Skip To Notification Chart'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#alertsummary-reaction-time">
-                    <?= t('Skip To Reaction Time Chart'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#alertsummary-trend">
-                    <?= t('Skip To Trend'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#alertsummary-top5">
-                    <?= t('Sktip To Top 5 Alerts'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#alertsummary-history">
-                    <?= t('Skip To Alert History'); ?>
-                </a>
-            </li>
-        </ul>
-    </div>
 
     <div class="hbox">
         <div class="hbox-item">
diff --git a/modules/monitoring/application/views/scripts/process/info.phtml b/modules/monitoring/application/views/scripts/process/info.phtml
index 12cb01fbc..639b5ebc0 100644
--- a/modules/monitoring/application/views/scripts/process/info.phtml
+++ b/modules/monitoring/application/views/scripts/process/info.phtml
@@ -12,26 +12,7 @@ $cp = $this->checkPerformance()->create($this->checkperformance);
     <h1 tabindex="-1">
         <?= $this->translate('Monitoring Health'); ?>
     </h1>
-    <div class="skip-links skip-links-inline">
-        <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-        <ul>
-            <li>
-                <a tabindex="0" href="#process-feature-command">
-                    <?= t('Skip To Feature Commands'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#process-process-info">
-                    <?= t('Skip To Process Information'); ?>
-                </a>
-            </li>
-            <li>
-                <a tabindex="0" href="#process-performance-info">
-                    <?= t('Skip To Performance Information'); ?>
-                </a>
-            </li>
-        </ul>
-    </div>
+
     <div class="boxview">
 
         <div class="box left">
@@ -98,26 +79,6 @@ $cp = $this->checkPerformance()->create($this->checkperformance);
             <h2 tabindex="-1" id="process-performance-info">
                 <?= $this->translate('Performance Info') ?>
             </h2>
-            <div class="skip-links skip-links-inline">
-                <h2 class="sr-only"><?= $this->translate('Skip Links'); ?></h2>
-                <ul>
-                    <li>
-                        <a tabindex="0" href="#process-object-summaries">
-                            <?= t('Skip To Object Summaries'); ?>
-                        </a>
-                    </li>
-                    <li>
-                        <a tabindex="0" href="#process-active-checks">
-                            <?= t('Skip To Active Checks'); ?>
-                        </a>
-                    </li>
-                    <li>
-                        <a tabindex="0" href="#process-passive-checks">
-                            <?= t('Skip To Passive Checks'); ?>
-                        </a>
-                    </li>
-                </ul>
-            </div>
 
             <h3 tabindex="-1" id="process-object-summaries">
                 <?= $this->translate('Object summaries') ?>