Skip to content

Commit

Permalink
Add a phase-banner class and update examples
Browse files Browse the repository at this point in the history
Keep the .phase-banner-alpha and .phase-banner-beta classes, so this
won’t be a breaking change, but since there isn’t a difference between
the two (as we’re not passing in a state to change the colour of the
phase tag), use a single class - phase-banner for both examples.
  • Loading branch information
gemmaleigh committed Nov 29, 2016
1 parent 3dab52b commit 32ca36b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion app/views/includes/phase_banner.html
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<div class="phase-banner-alpha">
<div class="phase-banner">
<p>
<strong class="phase-tag">ALPHA</strong>
<span>This is a new service – your <a href="https://designpatterns.hackpad.com/GOV.UK-elements-feedback-sKrDyQxcfA2">feedback</a> will help us to improve it.</span>
Expand Down
2 changes: 1 addition & 1 deletion app/views/snippets/phase_banner_alpha.html
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<div class="phase-banner-alpha">
<div class="phase-banner">
<p>
<strong class="phase-tag">ALPHA</strong>
<span>This is a new service – your <a href="#">feedback</a> will help us to improve it.</span>
Expand Down
6 changes: 2 additions & 4 deletions public/sass/elements/_phase-banner.scss
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
// Phase banners
// ==========================================================================

.phase-banner-alpha {
@include phase-banner();
}

.phase-banner,
.phase-banner-alpha,
.phase-banner-beta {
@include phase-banner();
}

0 comments on commit 32ca36b

Please sign in to comment.