User Groups - removed label: enabled/disabled users v2

[pe7er]

New PR to replace #12086 "User Groups - removed label: enabled/disabled users"
See #12086 for description.

In this new PR I just removed the "enabled users" and "disabled users" label, and only kept the icon.
Should work on mobile too now.

[ghost]

I have tested this item ✅ successfully on ee17910

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17924.}

[Quy]

It should be done like the Categories page where the text is wrapped in a <span class="element-invisible"> for accessiblity.

<span class="icon-publish hasTooltip" aria-hidden="true" title="Published items"><span class="element-invisible">Published items</span></span>

[ghost]

@pe7er any Opinion on @Quy's Comment above?

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17924.}

[pe7er]

@Quy Sorry, I do not understand your remark.
I just removed 2 labels with span tags around them.

[brianteeman]

@pe7er the problem is that by doing that you left the code in a non accessible state. The change suggested by @Quy will resolve that. For reference see http://fontawesome.io/accessibility/

[pe7er]

@brianteeman @Quy I don't understand what needs to be changed.
Could you please provide a patch. Thanks!

[brianteeman]

Tried to create a pull on your branch but it wouldnt let me ;( This is the code

						<th width="1%" class="nowrap center">
							<span class="icon-publish hasTooltip" aria-hidden="true" title="<?php echo JText::_('COM_USERS_COUNT_ENABLED_USERS'); ?>">
								<span class="element-invisible"><?php echo JText::_('COM_USERS_COUNT_ENABLED_USERS'); ?></span>
							</span>
						</th>
						<th width="1%" class="nowrap center">
							<span class="icon-unpublish hasTooltip" aria-hidden="true" title="<?php echo JText::_('COM_USERS_COUNT_DISABLED_USERS'); ?>">
								<span class="element-invisible"><?php echo JText::_('COM_USERS_COUNT_DISABLED_USERS'); ?></span>
							</span>
						</th>

[ladyjer]

I have tested this item ✅ successfully on ee17910

Hi, tested successfully.
I agree with Brian, element is needed.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17924.}

[ghost]

Ready to Commit after two successful tests.

[brianteeman]

Please remove the RTC it needs to be updated with the code I suggested

[ghost]

Status set back on "Pending" as stated above.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17924.}

[brianteeman]

@pe7er will you be updating this or shall we close it

[joomla-cms-bot]

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/17924

[Quy]

Closing in favor for #20279.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17924.}