? Pending

User tests: Successful: Unsuccessful:

avatar tonypartridge
tonypartridge
23 Nov 2017

Pull Request for Issue #18717 .

Summary of Changes

Removed old title text output
Added in new fieldset
Included name above the tabs
Follows Article Edit screen layout

Testing Instructions

Install, view / edit users.

avatar tonypartridge tonypartridge - open - 23 Nov 2017
avatar tonypartridge tonypartridge - change - 23 Nov 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Nov 2017
Category Administration com_users
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 23 Nov 2017 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 23 Nov 2017

I have tested this item ? unsuccessfully on e573908

Fieldset-Label:

bildschirmfoto 2017-11-23 um 14 06 16

Without PR:

bildschirmfoto 2017-11-23 um 14 08 10


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18824.
avatar tonypartridge tonypartridge - change - 23 Nov 2017
Labels Added: ?
avatar tonypartridge
tonypartridge - comment - 23 Nov 2017

Thanks @franz-wohlkoenig

Please re-test.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 23 Nov 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 23 Nov 2017

I have tested this item successfully on 86435a8


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

avatar tonypartridge
tonypartridge - comment - 24 Nov 2017

@brianteeman could you test pretty please?

avatar brianteeman
brianteeman - comment - 24 Nov 2017

its for j4 - i am not testing anything ui related for j4 until we have the actual ui that is going to be used.

avatar tonypartridge
tonypartridge - comment - 24 Nov 2017

Yep, but it’s not really theming UI related more UX and this would pass trough on all theming.

On 24 Nov 2017, 21:28 +0000, Brian Teeman notifications@github.com, wrote:

its for j4 - i am not testing anything ui related for j4 until we have the actual ui that is going to be used.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

avatar Quy
Quy - comment - 26 Nov 2017

For edit.php, use this instead.

<?php echo JLayoutHelper::render('joomla.edit.title_alias', $this); ?>
avatar tonypartridge
tonypartridge - comment - 27 Nov 2017

@Quy Now updated @franz-wohlkoenig please re-test.

avatar franz-wohlkoenig franz-wohlkoenig - change - 27 Nov 2017
Title
Move Users name above tabs on edit screen and remove 'Title' i.e username again.
[4.0] Move Users name above tabs on edit screen and remove 'Title' i.e username again.
avatar joomla-cms-bot joomla-cms-bot - edited - 27 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 27 Nov 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Nov 2017

I have tested this item successfully on 1bdeeee


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 27 Nov 2017
Easy No Yes
avatar tonypartridge
tonypartridge - comment - 25 Mar 2018

Good point @Quy now updated.

avatar tonypartridge
tonypartridge - comment - 25 Mar 2018

p.s. can we get re-tests.

avatar ggppdk
ggppdk - comment - 26 Mar 2018

I have tested this item successfully on 9ac523f


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

avatar ggppdk ggppdk - test_item - 26 Mar 2018 - Tested successfully
avatar ggppdk
ggppdk - comment - 26 Mar 2018

This needs rebasing so that unit tests will run properly ?

avatar tonypartridge
tonypartridge - comment - 27 Mar 2018

@ggppdk now down, can you test and also @franz-wohlkoenig if you get a minute :-)

avatar tonypartridge tonypartridge - change - 2 Apr 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-04-02 19:17:57
Closed_By tonypartridge
avatar tonypartridge tonypartridge - close - 2 Apr 2018
avatar tonypartridge
tonypartridge - comment - 2 Apr 2018

Will re do at some pointing, closing for now.

Add a Comment

Login with GitHub to post a comment