From 3c5592bdbc8f56e5a766071db66ebdcf830a7c39 Mon Sep 17 00:00:00 2001
From: ahamed07sajeeb <sajeeb07ahamed@gmail.com>
Date: Fri, 22 Nov 2019 12:42:14 +0600
Subject: [PATCH 1/2] Fixed searchtools toggle ordering button's discernible
 text issue, #56

---
 layouts/joomla/searchtools/default/list.php | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/layouts/joomla/searchtools/default/list.php b/layouts/joomla/searchtools/default/list.php
index 2f582dda319..f7bd36afbe7 100644
--- a/layouts/joomla/searchtools/default/list.php
+++ b/layouts/joomla/searchtools/default/list.php
@@ -38,7 +38,14 @@
 						</div>
 					</div>
 					<?php if($fieldName === 'list_fullordering') : ?>
-						<button type="button" class="btn btn-link js-stools-order-toggler hasTooltip" title="<?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?>"><i class="duotone icon-lg <?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? 'icon-descending' : 'icon-ascending'; ?>"></i></button>
+						<button type="button" 
+							class="btn btn-link js-stools-order-toggler hasTooltip" 
+							title="<?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?>"
+							aria-label="<?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?>"
+						>
+							<span class="sr-only"><?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?></span>
+							<span class="duotone icon-lg <?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? 'icon-descending' : 'icon-ascending'; ?>"></span>
+						</button>
 					<?php endif; ?>
 				</div>
 			<?php endif; ?>

From 9c1cdb401613a15018021a8401c75901edb1a04e Mon Sep 17 00:00:00 2001
From: ahamed07sajeeb <sajeeb07ahamed@gmail.com>
Date: Fri, 22 Nov 2019 12:47:40 +0600
Subject: [PATCH 2/2] Removed the element containing class sr-only, use
 aria-label instead

---
 layouts/joomla/searchtools/default/list.php | 1 -
 1 file changed, 1 deletion(-)

diff --git a/layouts/joomla/searchtools/default/list.php b/layouts/joomla/searchtools/default/list.php
index f7bd36afbe7..d19202bc236 100644
--- a/layouts/joomla/searchtools/default/list.php
+++ b/layouts/joomla/searchtools/default/list.php
@@ -43,7 +43,6 @@ class="btn btn-link js-stools-order-toggler hasTooltip"
 							title="<?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?>"
 							aria-label="<?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?>"
 						>
-							<span class="sr-only"><?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? Text::_('JGLOBAL_ORDER_DESCENDING_DIRN') : Text::_('JGLOBAL_ORDER_ASCENDING_DIRN'); ?></span>
 							<span class="duotone icon-lg <?php echo !empty($orderDirn) && strtoupper($orderDirn) === 'ASC' ? 'icon-descending' : 'icon-ascending'; ?>"></span>
 						</button>
 					<?php endif; ?>