Merge pull request #4443 from crydotsnake/feature/improve-user-sort-b…

…ehavior

FEATURE: Improve User sorting behavior in user management backend module

Neos.Neos/Resources/Private/Styles/Modules/Administration/_Users.scss

@@ -0,0 +1,3 @@
+.users-table thead th a:hover {
+	color: #{$blue}
+}

Neos.Neos/Resources/Private/Styles/Modules/_Modules.scss

@@ -2,6 +2,7 @@
   @import "Administration/Configuration";
   @import "Administration/Packages";
   @import "Administration/Sites";
+	@import "Administration/Users";
   @import "Management/Workspaces";
   @include font;
 

Neos.Neos/Resources/Private/Templates/Module/Administration/Users/Index.html

@@ -21,16 +21,16 @@
 
 		<f:if condition="{users}">
 			<f:widget.paginate objects="{users}" as="paginatedUsers">
-			<table class="neos-table">
+			<table class="neos-table users-table">
 					<thead>
 						<tr>
-							<th>
+							<th title="{neos:backend.translate(id: 'users.sortBy.fullName', source: 'Modules', package: 'Neos.Neos')}">
 								<f:render section="headerLabel" arguments="{sortDirection:sortDirection, sortBy:sortBy, sortingField:'name.fullName', labelIdentifier:'users.name'}" />
 							</th>
-							<th>
+							<th title="{neos:backend.translate(id: 'users.sortBy.accountsAndRoles', source: 'Modules', package: 'Neos.Neos')}">
 								<f:render section="headerLabel" arguments="{sortDirection:sortDirection, sortBy:sortBy, sortingField:'accounts.accountIdentifier', labelIdentifier:'users.accountsAndRoles'}" />
 							</th>
-							<th>
+							<th title="{neos:backend.translate(id: 'users.sortBy.lastLogin', source: 'Modules', package: 'Neos.Neos')}">
 								<f:render section="headerLabel" arguments="{sortDirection:sortDirection, sortBy:sortBy, sortingField:'accounts.lastSuccessfulAuthenticationDate', labelIdentifier:'users.lastLogin'}" />
 							</th>
 							<th>&nbsp;</th>
@@ -134,14 +134,14 @@
 				<f:then>
 					<f:link.action action="index" data="{neos-toggle: 'tooltip'}" arguments="{sortDirection: 'DESC', sortBy: sortingField}" addQueryString="TRUE">
 						<neos:backend.translate id="{labelIdentifier}" source="Modules" package="Neos.Neos"/>
+						<i class="fas fa-sort-amount-up"></i>
 					</f:link.action>
-					<i class="fas fa-caret-up"></i>
 				</f:then>
 				<f:else>
 					<f:link.action action="index" data="{neos-toggle: 'tooltip'}" arguments="{sortDirection: 'ASC', sortBy: sortingField}" addQueryString="TRUE">
 						<neos:backend.translate id="{labelIdentifier}" source="Modules" package="Neos.Neos"/>
+						<i class="fas fa-sort-amount-down"></i>
 					</f:link.action>
-					<i class="fas fa-caret-down"></i>
 				</f:else>
 			</f:if>
 		</f:then>

Neos.Neos/Resources/Private/Translations/en/Modules.xlf

@@ -421,7 +421,15 @@
 			<trans-unit id="users.electronicAddress.usage.type.Work" xml:space="preserve">
 				<source>Work</source>
 			</trans-unit>
-
+      <trans-unit id="users.sortBy.fullName" xml:space="preserve">
+				<source>Sort by first and last name</source>
+      </trans-unit>
+      <trans-unit id="users.sortBy.accountsAndRoles" xml:space="preserve">
+				<source>Sort by user accounts and roles</source>
+      </trans-unit>
+      <trans-unit id="users.sortBy.lastLogin" xml:space="preserve">
+				<source>Sort by last login</source>
+      </trans-unit>
 			<trans-unit id="packages.label" xml:space="preserve">
 				<source>Package Management</source>
 			</trans-unit>