Skip to content

Commit 5f7e16d

Browse files
authored
Merge pull request #998 from CVEProject/tat-233
#233 enable Newsletter Signup page, update styling and label
2 parents bf06847 + 60ea013 commit 5f7e16d

File tree

4 files changed

+27
-24
lines changed

4 files changed

+27
-24
lines changed

src/assets/style/globals.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ h6 {
395395
.cve-white-bg-gray-border-container {
396396
background-color: white;
397397
border: 1px solid $theme-color-base-lighter;
398-
padding: 0 1rem 2rem 1rem;
398+
padding: 0 1rem 1rem 1rem;
399399
margin-top: 1rem;
400400
}
401401

src/components/FooterModule.vue

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,7 @@
1313
<ul>
1414
<li><router-link to="/Media/News/AllNews">News</router-link></li>
1515
<li><router-link to="/Media/Events">Events</router-link></li>
16-
<li>
17-
<span class="icon-text">
18-
<a href="https://cve.mitre.org/news/newsletter.html" target="_blank">Sign up for e-newsletter
19-
<span class="icon cve-icon-xxs">
20-
<p id="enewsletter" class="is-hidden">external site</p>
21-
<font-awesome-icon icon="external-link-alt" aria-labelledby="enewsletter">
22-
</font-awesome-icon>
23-
</span>
24-
</a>
25-
</span>
26-
</li>
16+
<li><router-link to="/Media/NewsletterSignup">Sign up for e-newsletter</router-link></li>
2717
</ul>
2818
</div>
2919
<div class="column">

src/router/index.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import Podcasts from '@/views/Media/News/Podcasts.vue';
3232
import NewsItem from '@/views/Media/News/NewsItem.vue';
3333
import NewsArchives from '@/views/Media/News/NewsArchives.vue';
3434
import BlogArchives from '@/views/Media/News/BlogArchives.vue';
35+
import NewsletterSignup from '@/views/Media/News/NewsletterSignup.vue';
3536
import Events from '@/views/Media/Events.vue';
3637

3738
Vue.use(VueRouter);
@@ -302,6 +303,14 @@ const routes = [
302303
title: 'Blog Archives | CVE',
303304
},
304305
},
306+
{
307+
path: '/Media/NewsletterSignup',
308+
name: 'NewsletterSignup',
309+
component: NewsletterSignup,
310+
meta: {
311+
title: 'NewsletterSignup | CVE',
312+
},
313+
},
305314
{
306315
path: '/Media/Events',
307316
name: 'Events',

src/views/Media/News/NewsletterSignup.vue

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,13 @@
2525
</ul>
2626
</div>
2727
<div class="field is-horizontal">
28-
<label class="field-label cve-field-label">Email Address <span class="has-text-weight-normal">(Required)</span></label>
28+
<label class="field-label cve-field-label">Email Address <span class="is-size-7 has-text-weight-normal is-italic">Required</span></label>
2929
<div class="field-body control">
30-
<input v-model.trim="emailAddress" class="input" type="email" placeholder="Enter email address">
30+
<input v-model.trim="emailAddress" class="input" type="email" placeholder="Email address">
3131
</div>
3232
</div> <!-- end of field -->
3333
<div class="field is-horizontal">
34-
<label class="field-label cve-field-label">What is 9 + 4? <span class="has-text-weight-normal">(Required)</span></label>
34+
<label class="field-label cve-field-label">What is 9 + 4? <span class="is-size-7 has-text-weight-normal is-italic">Required</span></label>
3535
<div class="field-body control">
3636
<input v-model="mathAnswer" class="input" type="number" placeholder="Enter sum of 9 + 4">
3737
</div>
@@ -140,18 +140,18 @@
140140
<div class="field is-horizontal">
141141
<label class="field-label cve-field-label"></label>
142142
<div class="field-body control">
143-
<input type="submit" value="Subscribe" class="button cve-button cve-base-color">
143+
<button class="button cve-button cve-base-color">Subscribe</button>
144144
</div>
145145
</div> <!-- end of field -->
146146
</form>
147147
</div> <!-- end of container -->
148148
<div v-if="submitted" class="notification is-success">
149-
<h2 class="cve-top-margin">
149+
<div>
150150
<p id="successIcon" class="is-hidden">success</p>
151-
<font-awesome-icon style="flex: 0 0 40px;" size="1x" icon="check-circle" role="success"
152-
aria-labelledby="successIcon" aria-hidden="false" />
151+
<font-awesome-icon style="flex: 0 0 40px;" size="1x" icon="check-circle" role="success" aria-labelledby="successIcon"
152+
aria-hidden="false" />
153153
Success! Your subscription request has been submitted.
154-
</h2>
154+
</div>
155155
</div>
156156
<div class="mt-2">View our <router-link to=/Legal/PrivacyPolicy>Privacy Policy</router-link></div>
157157
</div>
@@ -193,13 +193,13 @@ export default {
193193
checkForm() {
194194
this.errors = [];
195195
if (!this.emailAddress) {
196-
this.errors.push('Email required.');
196+
this.errors.push('Email address required');
197197
} else if (!this.validEmail(this.emailAddress)) {
198-
this.errors.push('Valid email required.');
198+
this.errors.push('Valid email adress required');
199199
}
200200
201201
if (!this.mathAnswer) {
202-
this.errors.push('Response to 9 + 4 required');
202+
this.errors.push('Response to "What is 9 + 4?" required');
203203
}
204204
205205
if (!this.errors.length) {
@@ -217,7 +217,7 @@ export default {
217217
218218
</script>
219219
<style scoped lang="scss">
220-
@import '../../assets/style/globals.scss';
220+
@import '@/assets/style/globals.scss';
221221
222222
.cve-field-label {
223223
font-weight: bold;
@@ -226,4 +226,8 @@ export default {
226226
.cve-top-margin {
227227
margin-top: 20px;
228228
}
229+
230+
.field {
231+
margin-bottom: 8px !important;
232+
}
229233
</style>

0 commit comments

Comments
 (0)