? ? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
14 Mar 2018

Pull Request for Issue # .

Summary of Changes

Textarea (AKA editor none) implementing the Joomla API for editors

Testing Instructions

Install from this PR's branch. Change the default editor to none. Check that the buttons work and they do:

  • insert at the start of the text
  • insert at the end of the text
  • after making some selection and trying some of the buttons (the value from the xtd-button should replace the texture selection).

Mind that modals are not closing, it's a known issue!

Expected result

Actual result

Documentation Changes Required

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
3.00

avatar dgt41 dgt41 - open - 14 Mar 2018
avatar dgt41 dgt41 - change - 14 Mar 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Mar 2018
Category JavaScript Repository Layout Front End Plugins
avatar dgt41 dgt41 - change - 14 Mar 2018
Labels Added: ?
avatar dgt41 dgt41 - change - 15 Mar 2018
The description was changed
avatar dgt41 dgt41 - edited - 15 Mar 2018
avatar Samgithub1997
Samgithub1997 - comment - 19 Mar 2018

@dgt41 Are you talking about this editor
And if you are, what exactly are you trying to test? Closing the Modal?
Please help me test your patch

selection_025

avatar dgt41
dgt41 - comment - 19 Mar 2018

@Samgithub1997 nope, that's the default editor (tinyMCE) you need to go to Global Configuration and change the default editor to none:
screen shot 2018-03-19 at 21 07 07

Then try to insert, eg an image using the buttons bellow the editor...

avatar Samgithub1997
Samgithub1997 - comment - 19 Mar 2018

@dgt41 Thanks a lot!

avatar dgt41
dgt41 - comment - 19 Mar 2018

Nice, so if there is no difference (there shouldn't be) then this is a passing test. All that is happening with this PR is moving the javascript form the old version (eg DOMConentLoaded) to the brand new web components. There is no other enhancement here...

avatar Samgithub1997
Samgithub1997 - comment - 19 Mar 2018

Alright ! Thanks

avatar Anu1601CS
Anu1601CS - comment - 19 Mar 2018
avatar Samgithub1997 Samgithub1997 - test_item - 19 Mar 2018 - Tested successfully
avatar Samgithub1997
Samgithub1997 - comment - 19 Mar 2018

I have tested this item successfully on 0adcf61

The test was successful. It wasn't an issue but basically a necessary up-gradation from older JS version to a new web component.


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

avatar Samgithub1997 Samgithub1997 - test_item - 19 Mar 2018 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 19 Mar 2018

I have tested this item successfully on 0adcf61


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

avatar Anu1601CS Anu1601CS - test_item - 19 Mar 2018 - Tested successfully
avatar Quy Quy - change - 19 Mar 2018
Status Pending Ready to Commit
avatar joomla-cms-bot joomla-cms-bot - edited - 19 Mar 2018
avatar Quy
Quy - comment - 19 Mar 2018

RTC


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

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

I have tested this item successfully on b6c6486


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

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

@Samgithub1997 can you please retest?

avatar Samgithub1997
Samgithub1997 - comment - 5 Apr 2018

I've exam in two hours. I'll test it by today

avatar laoneo
laoneo - comment - 5 Apr 2018

If you can fix the conflict, then we are good to go here.

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 07:39:18
Closed_By laoneo
avatar laoneo laoneo - close - 5 Apr 2018
avatar laoneo laoneo - merge - 5 Apr 2018
avatar Samgithub1997
Samgithub1997 - comment - 5 Apr 2018

I have tested this item successfully on 2e1fcb4


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

avatar Samgithub1997 Samgithub1997 - test_item - 5 Apr 2018 - Tested successfully
avatar Samgithub1997
Samgithub1997 - comment - 5 Apr 2018

@laoneo , what conflict are you talking about ?

avatar laoneo
laoneo - comment - 6 Apr 2018

There was a conflict, so I couldn't merge it. Dimitris fixed it.

Add a Comment

Login with GitHub to post a comment