? ? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
11 Feb 2018

Pull Request for Issue #19427

Summary of Changes

  • Moving the olde code to ES6 and custom elements
  • Remove chosen (=== dead code)

Testing Instructions

Go to admin->modules, edit a module, change the position and check the order field

Expected result

Actual result

Documentation Changes Required

avatar dgt41 dgt41 - open - 11 Feb 2018
avatar dgt41 dgt41 - change - 11 Feb 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Feb 2018
Category Administration com_modules JavaScript Repository Layout
avatar Anu1601CS
Anu1601CS - comment - 13 Feb 2018

@dgt41 what, if none of the options are selected its shows like

screenshot from 2018-02-13 21-06-26

And this one is the previous image without the custom element.

screenshot from 2018-02-13 21-06-08

Also, is search option removed?

avatar dgt41
dgt41 - comment - 13 Feb 2018

Also, is search option removed?

Yes

avatar dgt41 dgt41 - change - 13 Feb 2018
Labels Added: ?
avatar Anu1601CS
Anu1601CS - comment - 15 Feb 2018

@dgt41 Please check my comments here and correct this :)

avatar dgt41
dgt41 - comment - 15 Feb 2018

@Anu1601CS good catch!

avatar Anu1601CS
Anu1601CS - comment - 15 Feb 2018

Thanks

avatar Anu1601CS
Anu1601CS - comment - 15 Feb 2018

@dgt41 All things are good.
But, is this correct?

screenshot from 2018-02-16 03-43-16

data-attr="class=" module-ajax-ordering""

avatar Anu1601CS Anu1601CS - test_item - 15 Feb 2018 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 15 Feb 2018

I have tested this item successfully on 2617b00


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

avatar Anu1601CS Anu1601CS - test_item - 15 Feb 2018 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 1 Mar 2018

@astridx Please test this.

avatar astridx
astridx - comment - 1 Mar 2018

Maybe I do not understand this PR right? The module field always shows only one entry, that is None. I have got three modules published on the position?

And the position field did not contain the text type or select a position

modules articles most read test administration

avatar dgt41
dgt41 - comment - 1 Mar 2018

@astridx what are you getting in the main repo (i mean without the code of this PR)?

avatar dgt41
dgt41 - comment - 1 Mar 2018

And the position field did not contain the text type or select a position

Yes that was functionality from chosen.js, we will eventually replicate it in the custom element, but right now it's not there yet

avatar Anu1601CS
Anu1601CS - comment - 1 Mar 2018

@dgt41 what for Ordering it always shows none ?

Without PR it shows
screenshot from 2018-03-01 23-03-20

avatar dgt41
dgt41 - comment - 1 Mar 2018

Ok, there's a bug here, will push some code later on

avatar dgt41
dgt41 - comment - 1 Mar 2018

@Anu1601CS @astridx it should be ok now

51a0135 1 Mar 2018 avatar dgt41 CS
6b02e45 1 Mar 2018 avatar dgt41 CS
avatar Anu1601CS
Anu1601CS - comment - 1 Mar 2018

All these things seem OK form my side. :)

avatar dgt41
dgt41 - comment - 1 Mar 2018

console.log removed

avatar Anu1601CS
Anu1601CS - comment - 1 Mar 2018

@dgt41 Thanks

avatar Anu1601CS
Anu1601CS - comment - 1 Mar 2018

I have tested this item successfully on 8031d28


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

avatar Anu1601CS Anu1601CS - test_item - 1 Mar 2018 - Tested successfully
avatar astridx
astridx - comment - 1 Mar 2018

I have tested this item successfully on 8031d28

When I assigned the branch, I first saw only None in the field module ordering. I then ran npm install</tt
modules articles most read test administration

and executed all node build commands. Then it worked.


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

avatar astridx astridx - test_item - 1 Mar 2018 - Tested successfully
avatar Quy Quy - change - 1 Mar 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 1 Mar 2018

RTC


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

avatar laoneo
laoneo - comment - 4 Apr 2018

Can you fix conflicts here?

avatar dgt41 dgt41 - change - 4 Apr 2018
Labels Added: ?
avatar dgt41
dgt41 - comment - 4 Apr 2018

@laoneo done

avatar Anu1601CS
Anu1601CS - comment - 4 Apr 2018

I have tested this item successfully on e2ac2c1


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

avatar Anu1601CS Anu1601CS - test_item - 4 Apr 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Apr 2018

@astridx can you please retest?

avatar laoneo laoneo - change - 5 Apr 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-04-05 06:01:48
Closed_By laoneo
avatar laoneo laoneo - close - 5 Apr 2018
avatar laoneo laoneo - merge - 5 Apr 2018

Add a Comment

Login with GitHub to post a comment