? Failure

User tests: Successful: Unsuccessful:

avatar pe7er
pe7er
20 Sep 2016

On the User Groups page is the layout of the columns Enabled users and Disabled users and their columns are IMHO not nice.

Summary of Changes

This PR removes the text labels Enabled users and Disabled users so that the layout looks better and more in consistent with the similar columns of the item counters on the Category page (Content > Categories)

article-categories

Testing Instructions

Before the PR:

backend: Users > User Groups

users-groups-enabled-disabled-users-before

After the PR:

backend: Users > User Groups

users-groups-enabled-disabled-users-after

avatar pe7er pe7er - open - 20 Sep 2016
avatar pe7er pe7er - change - 20 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Sep 2016
Category Administration Components
avatar joomla-cms-bot joomla-cms-bot - change - 20 Sep 2016
Labels Added: ?
avatar zero-24
zero-24 - comment - 20 Sep 2016

looks much better but maybe we can move that colum a bit left atleast on desktop? As there are now much unused space:
image

avatar RonakParmar RonakParmar - test_item - 20 Sep 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 20 Sep 2016

I have tested this item successfully on 612400b


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12086.

avatar brianteeman
brianteeman - comment - 20 Sep 2016

Not in favour of this change as it is right now a we still have the clumns enabled/disabled in the users view

avatar tomartailored tomartailored - test_item - 23 Sep 2016 - Tested successfully
avatar tomartailored
tomartailored - comment - 23 Sep 2016

I have tested this item successfully on 612400b

i have tested this item successfully


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12086.

avatar JamalTailored JamalTailored - test_item - 24 Sep 2016 - Tested successfully
avatar JamalTailored
JamalTailored - comment - 24 Sep 2016

I have tested this item successfully on 612400b


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12086.

avatar hans2103 hans2103 - test_item - 4 Nov 2016 - Tested unsuccessfully
avatar hans2103
hans2103 - comment - 4 Nov 2016

I have tested this item ? unsuccessfully on 612400b

With this patch applied the text "Enabled users" and "Disabled users" will disappear... but due to the classes "hidden-phone hidden-tablet" on the th element the table header will disappear too. Result in meaningless columns.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12086.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Jan 2017

I have tested this item ? unsuccessfully on 612400b

On smaller Devices the meaning of columns is not clear because some or all Icons are at
wrong position:
1
or gone
2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12086.
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 11 Jan 2017 - Tested unsuccessfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Apr 2017
The description was changed
Status Pending Needs Review
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2017
Category Administration Components Administration com_users Components
avatar joomla-cms-bot joomla-cms-bot - edited - 6 Apr 2017
avatar korneliusz401 korneliusz401 - test_item - 10 Aug 2017 - Tested successfully
avatar korneliusz401
korneliusz401 - comment - 10 Aug 2017

I have tested this item successfully on 612400b


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12086.

avatar brianteeman
brianteeman - comment - 9 Sep 2017

Can you look at resolving the conflicts so that this can be tested please. Otherwise based on the comments above this will be closed short!y

avatar pe7er
pe7er - comment - 10 Sep 2017

I've created a new PR for this: #17924 - User Groups - removed label: enabled/disabled users v2
This PR can be closed.

avatar pe7er pe7er - change - 10 Sep 2017
Status Needs Review Closed
Closed_Date 0000-00-00 00:00:00 2017-09-10 14:39:24
Closed_By pe7er
avatar pe7er pe7er - close - 10 Sep 2017

Add a Comment

Login with GitHub to post a comment