diff options
| author | Julius Härtl <github@jus.li> | 2017-10-04 16:09:29 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-10-04 16:09:29 +0200 |
| commit | 060eac40d868b55733fef0389b487ef64a211a4d (patch) | |
| tree | dc61c25d99401b8ec4864f10c732581a36a86326 /core/templates | |
| parent | 66b4efe4c6a0580791d8d8b4286fb2ef2962c120 (diff) | |
| parent | f7e6da9a8eac68cb0aa0fc9a593d3fb05e41fb89 (diff) | |
| download | nextcloud-server-060eac40d868b55733fef0389b487ef64a211a4d.tar.gz nextcloud-server-060eac40d868b55733fef0389b487ef64a211a4d.zip | |
Merge pull request #6700 from nextcloud/accessibility-fixes
Accessibility fixes
Diffstat (limited to 'core/templates')
| -rw-r--r-- | core/templates/layout.user.php | 32 |
1 files changed, 13 insertions, 19 deletions
diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php index 6b9db9389ba..5008392d2a3 100644 --- a/core/templates/layout.user.php +++ b/core/templates/layout.user.php @@ -33,29 +33,21 @@ <header role="banner"><div id="header"> <div class="header-left"> <a href="<?php print_unescaped(link_to('', 'index.php')); ?>" - id="nextcloud" tabindex="1"> + id="nextcloud"> <div class="logo logo-icon"> <h1 class="hidden-visually"> - <?php p($theme->getName()); ?> + <?php p($theme->getName()); ?> <?php p(!empty($_['application'])?$_['application']: $l->t('Apps')); ?> </h1> </div> </a> - <a href="#" class="header-appname-container menutoggle" tabindex="2"> - <h1 class="header-appname"> - <?php p(!empty($_['application'])?$_['application']: $l->t('Apps')); ?> - </h1> - <div class="icon-caret"></div> - </a> - <ul id="appmenu"> <?php foreach ($_['navigation'] as $entry): ?> <li data-id="<?php p($entry['id']); ?>" class="hidden"> <a href="<?php print_unescaped($entry['href']); ?>" - tabindex="3" <?php if ($entry['active']): ?> class="active"<?php endif; ?>> <img src="<?php print_unescaped($entry['icon'] . '?v=' . $_['versionHash']); ?>" - class="app-icon"/> + class="app-icon" alt="<?php p($entry['name']); ?>" /> <div class="icon-loading-small-dark" style="display:none;"></div> </a> @@ -78,7 +70,7 @@ <ul> <?php foreach($_['navigation'] as $entry): ?> <li data-id="<?php p($entry['id']); ?>"> - <a href="<?php print_unescaped($entry['href']); ?>" tabindex="3" + <a href="<?php print_unescaped($entry['href']); ?>" <?php if( $entry['active'] ): ?> class="active"<?php endif; ?>> <svg width="16" height="16" viewBox="0 0 16 16"> <defs><filter id="invert-<?php p($entry['id']); ?>"><feColorMatrix in="SourceGraphic" type="matrix" values="-1 0 0 0 1 0 -1 0 0 1 0 0 -1 0 1 0 0 0 1 0"></feColorMatrix></filter></defs> @@ -103,15 +95,15 @@ </label> <input id="searchbox" type="search" name="query" value="" required - autocomplete="off" tabindex="5"> - <button class="icon-close-white" type="reset"></button> + autocomplete="off"> + <button class="icon-close-white" type="reset"><span class="hidden-visually"><?php p($l->t('Reset search'));?></span></button> </form> <div id="contactsmenu"> - <div class="icon-contacts menutoggle"></div> + <div class="icon-contacts menutoggle" tabindex="0" role="link"></div> <div class="menu"></div> </div> <div id="settings"> - <div id="expand" tabindex="6" role="link" class="menutoggle"> + <div id="expand" tabindex="0" role="link" class="menutoggle"> <div class="avatardiv<?php if ($_['userAvatarSet']) { print_unescaped(' avatardiv-shown'); } else { print_unescaped('" style="display: none'); } ?>"> <?php if ($_['userAvatarSet']): ?> <img alt="" width="32" height="32" @@ -142,10 +134,12 @@ <div id="sudo-login-background" class="hidden"></div> <form id="sudo-login-form" class="hidden"> - <?php p($l->t('This action requires you to confirm your password:')); ?><br> - <input type="password" class="question" autocomplete="new-password" name="question" value=" <?php /* Hack against browsers ignoring autocomplete="off" */ ?>" + <label> + <?php p($l->t('This action requires you to confirm your password')); ?><br/> + <input type="password" class="question" autocomplete="new-password" name="question" value=" <?php /* Hack against browsers ignoring autocomplete="off" */ ?>" placeholder="<?php p($l->t('Confirm your password')); ?>" /> - <input class="confirm icon-confirm" title="<?php p($l->t('Confirm')); ?>" value="" type="submit"> + </label> + <input class="confirm" value="<?php p($l->t('Confirm')); ?>" type="submit"> </form> <div id="content-wrapper"> |