? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
17 Aug 2018

Pull Request for Issue # .

Summary of Changes

Gives tables matching styling to other modules in the cpanel.

Testing Instructions

Apply PR. Open cpanel

Before

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 Modules Administration Templates (admin)
avatar ciar4n ciar4n - change - 17 Aug 2018
Labels Added: ?
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 cd67a09


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

avatar brianteeman
brianteeman - comment - 17 Aug 2018

I find the complete lack of any "lines" make these tables very hard to read

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

I have tested this item successfully on cd67a09


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

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 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 brianteeman
brianteeman - comment - 17 Aug 2018

Please remove RTC - this does not match the design

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

Status back on pending as stated above.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

Status back on pending as stated above.


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

avatar ciar4n
ciar4n - comment - 17 Aug 2018

I find the complete lack of any "lines" make these tables very hard to read

True. As this is a global change I've created a separate pr.. #21665

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

@ciar4n should this PR be closed?

avatar brianteeman
brianteeman - comment - 17 Aug 2018

with #21665 this can be made rtc


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

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

I have tested this item successfully on cd67a09


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

avatar brianteeman
brianteeman - comment - 17 Aug 2018

I have tested this item successfully on cd67a09


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

Using PR #21660 and #21665 got Screenshot below which didn't look like joomla/40-backend-template#427 (Table Header)

screen shot 2018-08-17 at 12 22 46

avatar ciar4n
ciar4n - comment - 17 Aug 2018

joomla/40-backend-template#427 is for the article list, where this pr is for the cpanel.

The 'design' doesn't have a table header on these cpanel cards. If we insist on been strict to the design then that means removing the table header.

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 3 successful tests.

Thanks for Comment @ciar4n, haven't looked at Article-List/cPanel.

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 13:28: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 guys!

Add a Comment

Login with GitHub to post a comment