? 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