?
avatar andrepereiradasilva
andrepereiradasilva
2 Nov 2016

Steps to reproduce the issue

Go to /installation/index.php?view=defaultlanguage&lang=en and check the multilingual button colors.

Expected result

Not red. Red usually means error or danger.

Actual result

image

System information (as much as possible)

Latest staging.

Additional comments

None.

avatar andrepereiradasilva andrepereiradasilva - open - 2 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 2 Nov 2016
Labels Added: ?
avatar C-Lodder
C-Lodder - comment - 2 Nov 2016

need to use btn-group btn-group-reversed in the form field

avatar brianteeman brianteeman - change - 2 Nov 2016
Category Installation
avatar andrepereiradasilva
andrepereiradasilva - comment - 2 Nov 2016

i don't think so, "No" is not bad also, is just an option

avatar brianteeman
brianteeman - comment - 2 Nov 2016

its not that simple in the installation - the css is not present (neither
is btn-group-yesno)

another example of where we need to review the red/green

On 2 November 2016 at 23:18, Lodder notifications@github.com wrote:

need to use btn-group btn-group-revered in the form field


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

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 3 Nov 2016

i will take a closer look at this tomorrow - it will need new css in the installation template for certain

avatar brianteeman
brianteeman - comment - 4 Nov 2016

This is all done by JS and if you compare the relevant code in installation.js with template.js you will see the ordering etc is very different. Not just a case of adding a class to the xml


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

avatar brianteeman brianteeman - change - 4 Nov 2016
Status New Confirmed
avatar brianteeman
brianteeman - comment - 8 Nov 2016

I have tried and failed to resolve this

avatar infograf768
infograf768 - comment - 13 Nov 2016

If we agree that we should not use here Green and Red as they correspond to Positive and Negative in general in our UI, I propose to highlight the Option selected with a neutral color.

My patch is ready for this, if you agree. It overrides the bootstrap default only for these specific fields.
I chose Orange as neutral color. It could be a grey too.
Here is what I get:

btn

@andrepereiradasilva
@brianteeman
@C-Lodder

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 Nov 2016

IIRC @C-Lodder @dgt41 @ciar4n are working on solve that issue in 4.0.

avatar infograf768
infograf768 - comment - 13 Nov 2016

we don't need to wait for 4.0...

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 Nov 2016

right ...

avatar brianteeman
brianteeman - comment - 13 Nov 2016

3.7 is supported for at least two years we should not put things off that
can be fixed in 3.7

@infograf768 I think a shade of blue is more common for a neutral colour

avatar infograf768
infograf768 - comment - 13 Nov 2016

@brianteeman
Would you be kind enough to post here the # value for that blue?

avatar infograf768
infograf768 - comment - 13 Nov 2016

Would #6ca8ff be OK?

avatar brianteeman
brianteeman - comment - 13 Nov 2016

On my phone so hard to check but something like

http://www.color-hex.com/color/509cec

avatar infograf768
infograf768 - comment - 13 Nov 2016

I will use a shade of that.
screen shot 2016-11-13 at 15 17 32

avatar C-Lodder
C-Lodder - comment - 13 Nov 2016

Seems fine. Why is there a blinking cursor on the active button in your first screenshot?

avatar infograf768
infograf768 - comment - 13 Nov 2016

@C-Lodder
I have some Firefox add-ons that some time trigger something.

avatar C-Lodder
C-Lodder - comment - 13 Nov 2016

ah fair enough

avatar infograf768 infograf768 - change - 13 Nov 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-11-13 14:36:48
Closed_By infograf768
avatar joomla-cms-bot joomla-cms-bot - close - 13 Nov 2016
avatar joomla-cms-bot
joomla-cms-bot - comment - 13 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - close - 13 Nov 2016
avatar infograf768
infograf768 - comment - 13 Nov 2016

Closing as we have a patch #12879


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

Add a Comment

Login with GitHub to post a comment