-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
…age and updated behat test
The design asks for a filter that auto-submits without a submit button. We used to have the better exposed filters plugin on sportrelief. Do we want to use this again or should we do it ourselves without the plugin? |
135c420
to
8c576e3
Compare
As discuss with @Heleen-cr seems like that D8 port has issues with ajax: https://www.drupal.org/node/2776615 |
8c576e3
to
c1dd8bf
Compare
…ampaign into PLAT-543-news-landing-page
This reverts commit 7f641fdcf45a64926265637be53f47385b5b8c58.
This reverts commit 327a4b90eb451ca917e31789150e490ea69cbfc3.
…43-news-landing-page
} | ||
} | ||
@include breakpoint($screen-md) { | ||
.article-teaser__image, .article-teaser__video { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leave it for now, but best to stick each new selector onto a new line
.button { | ||
width: 100%; | ||
padding: 0.4em 1em 0.4em 1em; | ||
border: 1px solid #cccccc; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indents plz
@@ -75,7 +75,7 @@ | |||
margin: -50px auto 0; | |||
padding: 1px 0; | |||
background: $colour-white; | |||
@include breakpoint-gutters; | |||
max-width: calc(#{$max-width} - (2 * #{gutter()})); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we happy to be changing this, seeing as it's global for all promo pages?
…43-news-landing-page
…43-news-landing-page
Fixes https://jira.comicrelief.com/browse/PLAT-543
Changes proposed in this pull request
Manual deployment instructions
(don't show titles as the title of the page will be added in the body field of the landing page)