? Pending

User tests: Successful: Unsuccessful:

avatar bertmert
bertmert
14 Feb 2017

Pull Request for Issue #14068

Please see comments #14068 (comment) and #14068 (comment)

Summary of Changes

  • Global configuration. Removed class btn-group-reversed from field offline because of wrong(?) button coloring.

Testing Instructions

  • Install current staging.

  • Go to global configuration.

  • Test coloring of button Site Offline. Yes is red. No is green.

  • Apply patch

  • Test again.

  • Yes should be green and No should be red now.

avatar bertmert bertmert - open - 14 Feb 2017
avatar bertmert bertmert - change - 14 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Feb 2017
Category Administration com_config
avatar bertmert bertmert - change - 14 Feb 2017
The description was changed
avatar bertmert bertmert - edited - 14 Feb 2017
avatar bertmert bertmert - change - 14 Feb 2017
The description was changed
avatar bertmert bertmert - edited - 14 Feb 2017
avatar bertmert bertmert - change - 14 Feb 2017
The description was changed
avatar bertmert bertmert - edited - 14 Feb 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 Feb 2017

I have tested this item successfully on 78496ec


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 14 Feb 2017 - Tested successfully
avatar Bakual
Bakual - comment - 14 Feb 2017

That is by design. The idea is that the recommended setting is green (No = Site not offline).

avatar Bakual
Bakual - comment - 14 Feb 2017

#12709 was the PR which changed it.

avatar bertmert
bertmert - comment - 14 Feb 2017

Hmmm. When I compare several settings here and there then this seems a lonely spot where Green == Recommended.
Use URL Rewriting: Recommended?
Shared Sessions: Recommended?
Platform Specific Caching: ...
Enable FTP:
Enable Proxy:

From my point of view that's not consistent.

avatar Bakual
Bakual - comment - 15 Feb 2017

If you read the PR, he said himself that this is only one parameter and more should be done as well.

avatar bertmert
bertmert - comment - 15 Feb 2017

After reading the PR and this unfinished discussion (#12704) I still think that the class btn-group-reversed should be removed here until a clear, logical, consistent concept is to be seen. At the moment green for one single "No" is just looking wrong.

avatar joomdonation
joomdonation - comment - 19 Feb 2017

I think we should leave it as how it is. Yes should be Red in this case to warn admin if he puts the site offline.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Feb 2017

decision @rdeutz?

avatar rdeutz
rdeutz - comment - 22 Feb 2017

we don't change this at this point, we need a more solid concept for what is right or wrong. Seems to me a task for the UX team

avatar rdeutz rdeutz - change - 22 Feb 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-02-22 16:56:42
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 22 Feb 2017

Add a Comment

Login with GitHub to post a comment