? ? Pending

User tests: Successful: Unsuccessful:

avatar frankmayer
frankmayer
6 Jan 2017

Summary of Changes

  • Make columns centered, where it makes sense and looks nicer than left-aligned ones. For example version columns, single or maximum two-word columns

  • Also fixed CS on that file

Testing Instructions

  1. Go to Extensions -> Update. Make sure there is a list of updates to view.
  2. Leave this tab as it is, for comparison. Open a new tab, apply the patch and revisit the update screen.
  3. Compare and comment / review.

Code review: Due to CS fixes, the changes in this files are best viewed with a setting to ignore whitespaces and empty lines (Apply w=1 argument to files view if viewing on github).

Documentation Changes Required

Maybe to set some standards for formatting of columns.

Example:

Before:

screenshot-joomla-patch-tester-1 2017-01-07 19-51-21

After:

screenshot-joomla-patch-tester-1 2017-01-06 11-42-32

avatar frankmayer frankmayer - open - 6 Jan 2017
avatar frankmayer frankmayer - change - 6 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jan 2017
Category Administration com_installer
avatar frankmayer frankmayer - change - 7 Jan 2017
Title
Make columns centered, where it makes sense (in one file only for now, to get a discussion going
Make columns centered, where it makes sense (in one file only for now, to get a discussion going)
avatar frankmayer frankmayer - edited - 7 Jan 2017
avatar frankmayer frankmayer - change - 7 Jan 2017
The description was changed
avatar frankmayer frankmayer - edited - 7 Jan 2017
avatar coolcat-creations
coolcat-creations - comment - 5 Feb 2017

I have tested this item successfully on 4db3d97


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

avatar coolcat-creations coolcat-creations - test_item - 5 Feb 2017 - Tested successfully
avatar coolcat-creations
coolcat-creations - comment - 5 Feb 2017

tested successfully, watch out if you miss extension updates in latest staging, then change the version.php in libraries/cms/version/ in line 33 to 3.6 - revert this change after your test again.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Feb 2017

@coolcat-creations is another View to test? Didn't get extensions after changing const RELEASE = '3.7'; to const RELEASE = '3.6';.

avatar coolcat-creations
coolcat-creations - comment - 5 Feb 2017

Did you install some outdated extensions, too? be sure all update Sites are also enabled at /administrator/index.php?option=com_installer&view=updatesites

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Feb 2017

i'm looking for outdated extensions, thanks @coolcat-creations

avatar joomlamarco
joomlamarco - comment - 5 Feb 2017

patch works as it should


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

avatar zero-24 zero-24 - change - 5 Feb 2017
Labels Added: ?
avatar zero-24 zero-24 - change - 5 Feb 2017
Milestone Added:
Status Pending Ready to Commit
Labels Added: ?
avatar zero-24
zero-24 - comment - 5 Feb 2017

RTC based on two successfull tests. i have just converted the spaces to tabs in the last commit.´Thanks!


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

avatar rdeutz rdeutz - change - 8 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-08 14:05:03
Closed_By rdeutz
Labels
avatar rdeutz rdeutz - close - 8 Feb 2017
avatar rdeutz rdeutz - merge - 8 Feb 2017

Add a Comment

Login with GitHub to post a comment