? ? Pending

User tests: Successful: Unsuccessful:

avatar wojsmol
wojsmol
6 Jul 2018

Pull Request for Issue #20997

Summary of Changes

The Install button in Sample data module is incorrectly marked as a link.
See:

Testing Instructions

code review and check if install buttons in sape data module are working.

Expected result

Install button in sample data module is a button.

Actual result

Install button in sample data module is a link.

Documentation Changes Required

N/A

cc @zwiastunsw @chmst please test

avatar wojsmol wojsmol - open - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jul 2018
Category Modules Administration
avatar brianteeman brianteeman - change - 6 Jul 2018
Title
Convert link to a button in sabledata admin module
Convert link to a button in samledata admin module
avatar brianteeman brianteeman - edited - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
Title
Convert link to a button in samledata admin module
[4.0][a11n]Convert link to a button in sampele data admin module
avatar wojsmol wojsmol - edited - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
Title
[4.0][a11n]Convert link to a button in sampele data admin module
[4.0][a11y]Convert link to a button in sampele data admin module
avatar wojsmol wojsmol - edited - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
The description was changed
avatar wojsmol wojsmol - edited - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
Title
[4.0][a11y]Convert link to a button in sampele data admin module
[4.0][a11y]Convert link to a button in sample data admin module
avatar wojsmol wojsmol - edited - 6 Jul 2018
avatar wojsmol wojsmol - change - 6 Jul 2018
The description was changed
avatar wojsmol wojsmol - edited - 6 Jul 2018
avatar SharkyKZ
SharkyKZ - comment - 6 Jul 2018

Add type="button" attribute.

avatar wojsmol wojsmol - change - 6 Jul 2018
Labels Added: ?
avatar zwiastunsw
zwiastunsw - comment - 6 Jul 2018

I have tested this item successfully on ea9fe6b


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

avatar zwiastunsw zwiastunsw - test_item - 6 Jul 2018 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 6 Jul 2018

I have tested this item successfully on ea9fe6b


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

avatar SharkyKZ SharkyKZ - test_item - 6 Jul 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Jul 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Jul 2018

Ready to Commit after two successful tests.

avatar wojsmol
wojsmol - comment - 10 Jul 2018
avatar brianteeman
brianteeman - comment - 11 Jul 2018

@wojsmol patience is a virtue

avatar wilsonge wilsonge - change - 11 Jul 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-07-11 07:31:33
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 11 Jul 2018
avatar wilsonge wilsonge - merge - 11 Jul 2018
avatar wilsonge
wilsonge - comment - 11 Jul 2018

Sorry for the delay in merging. Thanks!

Add a Comment

Login with GitHub to post a comment