raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-19 00:27:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Mark tom-select as changed

Browse Source 
Thankfully I was able to use basic DOM features, so there's no coupling of the logic with tom-select.

It wasn't going to be simple to get tom-select to listen for the 'changed' class on the original select, so I found a simple solution with a CSS sibling selector instead.
This commit is contained in:
David Cook
2024-02-29 12:20:50 +11:00
parent af748158aa
commit bfd6319cf2
3 changed files with 49 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
spec/javascripts/stimulus/bulk_form_controller_test.js
View File

@@ -36,6 +36,10 @@ describe("BulkFormController", () => {
<div data-record-id="1">
<input id="input1a" type="text" value="initial1a">
<input id="input1b" type="text" value="initial1b">
<select id="select1">
<option>one</option>
<option selected>two</option>
</select>
<button>a button is counted as a form element, but value is undefined</button>
</div>
<div data-record-id="2">
@@ -47,7 +51,7 @@ describe("BulkFormController", () => {
});
describe("marking changed fields", () => {
it("onInput", () => {
it("input: onInput", () => {
input1a.value = 'updated1a';
input1a.dispatchEvent(new Event("input"));
// Expect only first field to show changed
@@ -59,7 +63,23 @@ describe("BulkFormController", () => {
input1a.value = 'initial1a';
input1a.dispatchEvent(new Event("input"));
expect(input1a.classList).not.toContain('changed');
});
it("select: onInput", () => {
// Select a different option (it's the only way in Jest..)
select1.options[0].selected = true;
select1.options[1].selected = false;
select1.dispatchEvent(new Event("input"));
// Expect select to show changed
expect(input1a.classList).not.toContain('changed');
expect(input1b.classList).not.toContain('changed');
expect(select1.classList).toContain('changed');
// Change back to original value
select1.options[0].selected = false;
select1.options[1].selected = true;
select1.dispatchEvent(new Event("input"));
expect(select1.classList).not.toContain('changed');
});
it("multiple fields", () => {