? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
29 Aug 2017

Pull Request for Issue # .

Summary of Changes

  • Move the user field to web components
  • Original functionality
  • still jQuery until we move the modals to web components

Testing Instructions

Edit an article and change the Created By or Modified By fields

Expected result

Same functionality as 3.x

Actual result

Same functionality as 3.x

Documentation Changes Required

NOPE

avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2017
Category Layout JavaScript
avatar dgt41 dgt41 - open - 29 Aug 2017
avatar dgt41 dgt41 - change - 29 Aug 2017
Status New Pending
avatar dgt41 dgt41 - change - 29 Aug 2017
Labels Added: ?
avatar dgt41 dgt41 - change - 29 Aug 2017
Title
[4.0] webcomponents everywhere, media field
[4.0] webcomponents everywhere, user field
avatar dgt41 dgt41 - edited - 29 Aug 2017
avatar brianteeman
brianteeman - comment - 29 Aug 2017

Cant test as I get this error whenever I click on the user

Notice: Undefined variable: isMoo in C:\Apache24\htdocs\cms4\administrator\components\com_users\tmpl\users\modal.php on line 76
> Super User
avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2017
Category Layout JavaScript Administration com_users Layout JavaScript
avatar brianteeman
brianteeman - comment - 29 Aug 2017

No mootools error now and it seems to work ok. BUT the console has lots of errors related to flying-focus.js Not sure if thats related to this PR or not?

avatar dgt41
dgt41 - comment - 30 Aug 2017

@brianteeman there are two kinds of errors:

  • flying focus
  • missing source map file of popper.js
    but both are not related to this PR (you can move to a list view and still get those errors and list views don't have use input fields).
avatar brianteeman brianteeman - test_item - 30 Aug 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 Aug 2017

I have tested this item successfully on 0b46111


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

avatar C-Lodder
C-Lodder - comment - 30 Aug 2017

Hmm I'll get that error fixed for flying focus and release an update

avatar brianteeman
brianteeman - comment - 30 Aug 2017

thanks @C-Lodder

avatar dgt41 dgt41 - change - 17 Nov 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-11-17 09:57:33
Closed_By dgt41
avatar dgt41 dgt41 - close - 17 Nov 2017

Add a Comment

Login with GitHub to post a comment