? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
28 Aug 2017

Pull Request for Issue # .

Summary of Changes

  • Move the media field to web components
  • Original functionality
  • Provides more freedom to do do something more with this ultra standard Bootstrap field

REQUESTING FOR SOME BETTER DESIGN

If any designer comes along this PR and they feel that this part needs some redesign love, please do help and provide some mockups

Testing Instructions

Edit an article and change the intro and full text images

Expected result

Same functionality as 3.x

Actual result

screen shot 2017-08-28 at 11 18 32

screen shot 2017-08-28 at 11 18 58

screen shot 2017-08-28 at 11 18 48

Documentation Changes Required

NOPE

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

Looks like you have a bug ;)
screenshotr14-43-27

avatar dgt41
dgt41 - comment - 29 Aug 2017

It's this line: type="image" <? // @TODO add this attribute to the field in order to use it for all media types ?> should be type="image" <? /* @TODO add this attribute to the field in order to use it for all media types */ ?>

avatar brianteeman brianteeman - test_item - 29 Aug 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 29 Aug 2017

I have tested this item successfully on e65c4c1


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

avatar Fedik
Fedik - comment - 30 Sep 2017

the field seems works, but I cannot see any image in modal,
maybe because com_media changes

avatar dgt41
dgt41 - comment - 8 Oct 2017

This is already merged, from the new media manager repo

avatar dgt41 dgt41 - change - 8 Oct 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-10-08 09:35:58
Closed_By dgt41
avatar dgt41 dgt41 - close - 8 Oct 2017

Add a Comment

Login with GitHub to post a comment