Skip to content

Commit 89d5114

Browse files
committed
clear qualifier on subscription type change
1 parent e94feee commit 89d5114

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

app/assets/javascripts/subscriptions.js

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,18 @@
11
document.addEventListener('DOMContentLoaded', () => {
2+
const clearQualifier = () => {
3+
const qualifierField = document.querySelector('.js-subscription-qualifier-field');
4+
5+
if (qualifierField instanceof HTMLInputElement) {
6+
qualifierField.value = '';
7+
}
8+
};
9+
210
document.querySelectorAll('.js-subscription-type-select, .js-subscription-frequency-select').forEach((el) => {
3-
$(el).select2();
11+
$(el).select2().on('change', ($event) => {
12+
if ($event.target.matches('.js-subscription-type-select')) {
13+
clearQualifier();
14+
}
15+
});
416
});
517

618
$('.js-enable-subscription').on('change', async (evt) => {

app/views/subscriptions/new.html.erb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
<%= f.label :qualifier, class: "form-element" %>
2222
<div class="form-caption">Should be an id for user subscriptions or name for tag & category subscriptions.
2323
Leave the field blank otherwise.</div>
24-
<%= f.text_field :qualifier, class: 'form-element', value: @subscription.qualifier %>
24+
<%= f.text_field :qualifier,
25+
class: 'form-element js-subscription-qualifier-field',
26+
value: @subscription.qualifier %>
2527
</div>
2628

2729
<div class="form-group">

0 commit comments

Comments
 (0)