modern skin: download button uses hard-coded colours

[restena-sw]

Issue type

• Defect - Unexpected behaviour (obvious or has been verified by a project member).

Defect/Feature description

While it is possible to configure thematic colours for the product (config/Master.php and skins/modern/resources/css/cat-user.css), those colours are not considered for rendering the big download button. That one uses hardcoded eduroam-themed colours.

How to reproduce issue

1. Install product
2. change thematic colours at the pertinent places
3. observe the eduroam-blue download button

Detail of issue

This appears to happen in cat-user.css

button.large_button

[twoln]

The eduroam download button is controlled by the guess_os class setting. Hovever the background of the optional OpenRoaming button was indeed hard coded in the JS source. Changed this to be identical to the eduroam button - this both are controlled by the guess_os setting.

[restena-sw]

Hm, I didn't even realise /these/ buttons yet. I am talking about the big (blue) button on the front page...

[twoln]

I see no problem with this one - just change the background-color for large_button

[twoln]

Since we now have the css as php we could easily use php constants but this might narrow the possibilities for changes and so I did not make this large change at the moment.

[restena-sw]

It's not even that complicated. The user CSS at https://github.com/GEANT/CAT/blob/master/web/skins/modern/resources/css/cat-user.css#L21 defines color1..3 constants. Those could be referred to, which already makes the hard-coding go away.

[twoln]

I put in those constants myself and then stopped using them after realizing that some browsers had problems with them. Not sure wha the situation is now, you using php would be absolutely safe, the question is dow we think this is the correct approach.