Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distinguish between Whitehall environments #1557

Merged
merged 3 commits into from Jun 12, 2014
Merged

Distinguish between Whitehall environments #1557

merged 3 commits into from Jun 12, 2014

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jun 11, 2014

  • Distinguish between Preview and Production so that users don't accidentally make changes to Production content whilst under the assumption they're working on Preview.
  • Highlight environment using a favicon and a coloured label
  • Will need changes to deploy scripts (see https://github.gds/gds/alphagov-deployment/pull/603)

screen shot 2014-06-11 at 16 02 24

fofr added 3 commits Jun 11, 2014
* Show a red, orange or grey message depending on environment
* Message set at deploy time by an initialiser, following the same
patterns as publisher
* At thin widths the main nav collapses down and the margins are
reduced, use a media query to keep them consistent
* Use the Bootstrap 2 breakpoint specified in
http://getbootstrap.com/2.3.2/scaffolding.html#responsive
@alext
Copy link
Contributor

@alext alext commented Jun 11, 2014

I'll pick up the deployment change for you... I'm going to try to reduce some of the duplication in there at the same time.

alext added a commit that referenced this pull request Jun 12, 2014
Distinguish between Whitehall environments
@alext alext merged commit 09388e6 into master Jun 12, 2014
1 check passed
1 check passed
default Build #3203 succeeded on Jenkins
Details
@alext alext deleted the environment-indicator branch Jun 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.