? ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
5 Mar 2019

Pull Request for Issue #23965 .

Summary of Changes

Someone removed the / from the <hr> element and thus broke the plugin...

Testing Instructions

Follow the description on the issue

Expected result

screenshot 2019-03-05 at 15 57 21

Actual result

No tabs!

Documentation Changes Required

No, bug fix

avatar dgrammatiko dgrammatiko - open - 5 Mar 2019
avatar dgrammatiko dgrammatiko - change - 5 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Mar 2019
Category Front End Plugins
avatar dgrammatiko dgrammatiko - change - 5 Mar 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 5 Mar 2019
Category Front End Plugins Administration com_plugins Front End Plugins
avatar infograf768
infograf768 - comment - 7 Mar 2019

You are also modifying module long description.
Should it not be done in a different PR?

avatar dgrammatiko
dgrammatiko - comment - 7 Mar 2019

Should it not be done in a different PR?

Actually that was the issue this PR was supposed to fix. I misread the description of the issue and solved another problem. Then I checked again the issue and solved the initial issue. I'm happy if you can take the code and make 2 PRs out of this...

avatar flyingwombats flyingwombats - test_item - 7 Mar 2019 - Tested successfully
avatar flyingwombats
flyingwombats - comment - 7 Mar 2019

I have tested this item successfully on 856d79b

Does as it says. Click on the Show full description ... link and with patch you are taken to the Description tag, without patch it just stays on the Plugin tab and does not change to the description tab


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

avatar flyingwombats
flyingwombats - comment - 7 Mar 2019

I have tested this item successfully on 856d79b

Does as it says. Click on the Show full description ... link and with patch you are taken to the Description tag, without patch it just stays on the Plugin tab and does not change to the description tab


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

avatar Bodge-IT
Bodge-IT - comment - 9 Mar 2019

I have tested this item successfully on 856d79b

Description in PR #23965 is more accurate and what I tested against.

'Show description' link did nothing before patch, although I could see the tab and click on it. After patch, 'Show hescription' link sent me to description tab as one would expect.


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

avatar Bodge-IT
Bodge-IT - comment - 9 Mar 2019

I have tested this item successfully on 856d79b

Description in PR #23965 is more accurate and what I tested against.

'Show description' link did nothing before patch, although I could see the tab and click on it. After patch, 'Show hescription' link sent me to description tab as one would expect.


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

avatar Bodge-IT Bodge-IT - test_item - 9 Mar 2019 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 9 Mar 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 9 Mar 2019

Status "Ready To Commit".

avatar infograf768 infograf768 - change - 10 Mar 2019
Labels Added: ?
avatar wilsonge wilsonge - change - 15 Mar 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-03-15 13:44:49
Closed_By wilsonge
avatar wilsonge wilsonge - close - 15 Mar 2019
avatar wilsonge wilsonge - merge - 15 Mar 2019
avatar wilsonge
wilsonge - comment - 15 Mar 2019

Thanks!

Add a Comment

Login with GitHub to post a comment