? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
17 Aug 2018

Pull Request for Issue # .

Summary of Changes

To keep the workflow component inline with other components, the PR remove the table-striped class from tables

Testing Instructions

Apply PR. Navigate to workflows.

Brefore

image

After

image

Documentation Changes Required

avatar ciar4n ciar4n - open - 17 Aug 2018
avatar ciar4n ciar4n - change - 17 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2018
Category Administration com_workflow
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 17 Aug 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

I have tested this item successfully on c8115d5


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

avatar brianteeman
brianteeman - comment - 17 Aug 2018

I find it very hard to read a table that has no "lines"

avatar brianteeman
brianteeman - comment - 17 Aug 2018

The template design as proposed joomla/40-backend-template#427

Has clearly defined lines to demarcate the rows

avatar brianteeman brianteeman - test_item - 17 Aug 2018 - Tested successfully
avatar brianteeman
brianteeman - comment - 17 Aug 2018

I have tested this item successfully on c8115d5

with #21665 this is good to go


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 17 Aug 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

Ready to Commit after two successful tests.

avatar wilsonge wilsonge - change - 17 Aug 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-17 14:26:08
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 17 Aug 2018
avatar wilsonge wilsonge - merge - 17 Aug 2018
avatar wilsonge
wilsonge - comment - 17 Aug 2018

thanks!

avatar ciar4n
ciar4n - comment - 17 Aug 2018

Thanks for the tests!

Add a Comment

Login with GitHub to post a comment