diff options
| author | Jan C. Borchardt <hey@jancborchardt.net> | 2021-04-20 13:51:40 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-20 13:51:40 +0200 |
| commit | 9d1e2c50823fc2a2a1763e1153c0c18bce8fbddd (patch) | |
| tree | 3893c434819f57b2213f33ac0c8247365b48b768 /apps/files | |
| parent | 67ab4dd6c7b4ec51047bc128a5bdd5af1c36b100 (diff) | |
| parent | 738ac612d2535b8fb0d9cbc705937df78771f6af (diff) | |
| download | nextcloud-server-9d1e2c50823fc2a2a1763e1153c0c18bce8fbddd.tar.gz nextcloud-server-9d1e2c50823fc2a2a1763e1153c0c18bce8fbddd.zip | |
Merge pull request #26548 from nextcloud/design/files-accessibility
Files & Core accessibility fixes
Diffstat (limited to 'apps/files')
| -rw-r--r-- | apps/files/css/files.scss | 28 | ||||
| -rw-r--r-- | apps/files/templates/appnavigation.php | 5 | ||||
| -rw-r--r-- | apps/files/templates/index.php | 1 |
3 files changed, 26 insertions, 8 deletions
diff --git a/apps/files/css/files.scss b/apps/files/css/files.scss index dae10e343d3..66e2f2af1dd 100644 --- a/apps/files/css/files.scss +++ b/apps/files/css/files.scss @@ -476,10 +476,18 @@ table td.selection { } /* Use label to have bigger clickable size for checkbox */ -#fileList tr td.selection>.selectCheckBox + label, -.select-all + label { - padding: 16px; +#fileList tr td.selection>.selectCheckBox, +.select-all { + & + label { + padding: 16px; + } + + &:focus + label { + background-color: var(--color-background-hover); + border-radius: var(--border-radius-pill); + } } + #fileList tr td.selection>.selectCheckBox:focus-visible + label, .select-all:focus-visible + label { outline-offset: 0px; @@ -639,7 +647,11 @@ a.action > img { } } &:hover, &:focus { - opacity: .7; + opacity: 1; + } + &:focus { + background-color: var(--color-background-hover); + border-radius: var(--border-radius-pill); } } .fileActionsMenu a.action, a.action.action-share.shared-style { @@ -690,12 +702,14 @@ a.action > img { } .summary { - opacity: .3; + color: var(--color-text-maxcontrast); /* add whitespace to bottom of files list to correctly show dropdowns */ height: 250px; } #filestable .filesummary { width: 100%; + /* Width of checkbox and file preview */ + padding-left: 101px; } /* Less whitespace needed on link share page * as there is a footer and action menus have fewer entries. @@ -714,8 +728,8 @@ table tr.summary td { vertical-align: top; padding-top: 20px; } -.summary .info { - margin-left: 2px; +.summary td:first-child { + padding: 0; } .hiddeninfo { white-space: pre-line; diff --git a/apps/files/templates/appnavigation.php b/apps/files/templates/appnavigation.php index 8e654567946..7946d2a138a 100644 --- a/apps/files/templates/appnavigation.php +++ b/apps/files/templates/appnavigation.php @@ -93,7 +93,10 @@ function NavigationListElements($item, $l, $pinned) { NavigationElementMenu($item); if (isset($item['sublist'])) { ?> - <button class="collapse app-navigation-noclose" <?php if (sizeof($item['sublist']) == 0) { ?> style="display: none" <?php } ?>></button> + <button class="collapse app-navigation-noclose" + aria-label="<?php p($l->t('Toggle %1$s sublist', $item['name'])) ?>" + <?php if (sizeof($item['sublist']) == 0) { ?> style="display: none" <?php } ?>> + </button> <ul id="sublist-<?php p($item['id']); ?>"> <?php foreach ($item['sublist'] as $item) { diff --git a/apps/files/templates/index.php b/apps/files/templates/index.php index 8b6e2b0b91b..1e8b7cf1ac9 100644 --- a/apps/files/templates/index.php +++ b/apps/files/templates/index.php @@ -4,6 +4,7 @@ <?php if (!$_['isIE']) { ?> <input type="checkbox" class="hidden-visually" id="showgridview" + aria-label="<?php p($l->t('Toggle grid view'))?>" <?php if ($_['showgridview']) { ?>checked="checked" <?php } ?>/> <label id="view-toggle" for="showgridview" class="button <?php p($_['showgridview'] ? 'icon-toggle-filelist' : 'icon-toggle-pictures') ?>" title="<?php p($l->t('Toggle grid view'))?>"></label> |