Tried to make settings/apps more keyboard/screenreader friendly.

diff --git a/core/css/styles.css b/core/css/styles.css
index e3d8403e7502ea26d141beacc31b1a547dd214eb..c77df7d4cca22bdc36665fa201b5f0249edd73bb 100644 (file)
--- a/core/css/styles.css
+++ b/core/css/styles.css
@@ -62,6 +62,7 @@ input[type="submit"].highlight{ background:#ffc100; border:1px solid #db0; text-
 #leftcontent li:hover, #leftcontent li:active, #leftcontent li.active, .leftcontent li:hover, .leftcontent li:active, .leftcontent li.active { background:#eee; }
 #leftcontent li.active, .leftcontent li.active { font-weight:bold; }
 #leftcontent li:hover, .leftcontent li:hover { color:#333; background:#ddd; }
+#leftcontent a { height: 100%; display: block; margin: 0; padding: 0 1em 0 0; float: left; }
 #rightcontent, .rightcontent { position:fixed; top: 6.4em; left: 32.5em; overflow: auto }
 
 

diff --git a/settings/js/apps.js b/settings/js/apps.js
index df5300911aa388ffc404165644d0fe196f44f6c4..c15f0f30473659ac729e94db6e0db60784272e73 100644 (file)
--- a/settings/js/apps.js
+++ b/settings/js/apps.js
@@ -9,6 +9,12 @@ $(document).ready(function(){
                var app=$.parseJSON($(this).children('span').text());
                $(li).data('app',app);
        });
+       $('#leftcontent li').keydown(function(event) {
+               if (event.which == 13 || event.which == 32) {
+                       $(event.target).click()
+               }
+               return false;
+       });
        $('#leftcontent li').click(function(){
                var app=$(this).data('app');
                $('#rightcontent p').show();
@@ -25,6 +31,7 @@ $(document).ready(function(){
                $('#rightcontent input.enable').val((app.active)?t('settings','Disable'):t('settings','Enable'));
                $('#rightcontent input.enable').data('appid',app.id);
                $('#rightcontent input.enable').data('active',app.active);
+               return false;
        });
        $('#rightcontent input.enable').click(function(){
                var app=$(this).data('appid');

diff --git a/settings/templates/apps.php b/settings/templates/apps.php
index d25ca1bc7fb881f291288643a46b7df9a194f52a..7642ab7fb89c3e8d30f7b2b004b02b1dd5b8f64e 100644 (file)
--- a/settings/templates/apps.php
+++ b/settings/templates/apps.php
@@ -10,7 +10,7 @@
 <ul id="leftcontent">
        <?php foreach($_['apps'] as $app):?>
        <li <?php if($app['active']) echo 'class="active"'?> data-id="<?php echo $app['id'] ?>">
-               <?php  echo $app['name'] ?>
+               <a href="?appid=<?php echo $app['id'] ?>"><?php  echo $app['name'] ?></a>
                <span class="hidden">
                        <?php OC_JSON::encodedPrint($app,false) ?>
                </span>