? ? Success

User tests: Successful: Unsuccessful:

avatar MATsxm
MATsxm
8 Dec 2016

Summary of Changes

changing en-US color to default en-GB colour

Testing Instructions

on review

Documentation Changes Required

none

avatar MATsxm MATsxm - open - 8 Dec 2016
avatar MATsxm MATsxm - change - 8 Dec 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Dec 2016
Category Administration Language & Strings
avatar brianteeman
brianteeman - comment - 8 Dec 2016

I am a little torn on this one. On the one hand you are perfectly correct and it should have a U. On the other hand it is referring to a code element called color so it is correct without the U.

(and to make it more fun we already use the U for a similar field in the template styles)

avatar Hils
Hils - comment - 8 Dec 2016

If it code it should be background-color but if it is narrative it should be Background colour, in my view

avatar brianteeman
brianteeman - comment - 8 Dec 2016

So which one is it Hils if it is text describing code?

avatar ot2sen
ot2sen - comment - 8 Dec 2016

When it is readable text for the user then 'colour' is to be used as
suggested in the PR.
Had it been a code value/text that the user had to enter in for example a
style override then it would need to use code standard 'color'

On Thu, Dec 8, 2016 at 1:50 PM, Brian Teeman notifications@github.com
wrote:

So which one is it Hils if it is text describing code?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#13129 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwPrYEvPiNOinq-yC-UQ0JIO0Qj6gphks5rF_z4gaJpZM4LHuVd
.

avatar Hils
Hils - comment - 8 Dec 2016

It's just describing what happens if a button is pushed isn't it? If so colour should be the way to go in Joomla.

avatar brianteeman
brianteeman - comment - 8 Dec 2016

Ok. Fine by me. As I said I was undecided on this one I am happy to go with
the majority

On 8 Dec 2016 1:15 p.m., "Hils" notifications@github.com wrote:

It's just describing what happens if a button is pushed isn't it? If so
colour should be the way to go in Joomla.


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#13129 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8e0Q5_XaoFWVdRydC8vH28RUSLkkks5rGAMDgaJpZM4LHuVd
.

avatar jeckodevelopment
jeckodevelopment - comment - 8 Dec 2016

I have tested this item successfully on c2b8405


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

avatar jeckodevelopment jeckodevelopment - test_item - 8 Dec 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 8 Dec 2016

I have tested this item successfully on c2b8405


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

avatar andrepereiradasilva andrepereiradasilva - test_item - 8 Dec 2016 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 8 Dec 2016
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 8 Dec 2016

RTC


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

avatar jeckodevelopment jeckodevelopment - change - 8 Dec 2016
Milestone Added:
avatar wilsonge wilsonge - change - 8 Dec 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-12-08 17:07:01
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 8 Dec 2016
avatar wilsonge wilsonge - merge - 8 Dec 2016
avatar wilsonge wilsonge - reference | 7e1d3d6 - 8 Dec 16
avatar wilsonge wilsonge - merge - 8 Dec 2016
avatar wilsonge wilsonge - close - 8 Dec 2016

Add a Comment

Login with GitHub to post a comment