? ? 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