Prevent special characters from breaking notes search

jest.setup.js

@@ -2,6 +2,7 @@ import 'cross-fetch/polyfill';
 import server from './test-integration/mocks/server';
 
 window.HTMLElement.prototype.scrollIntoView = () => {};
+window.scrollTo = () => {};
 
 beforeAll(() => server.listen());
 afterEach(() => server.resetHandlers());

package.json

@@ -1,6 +1,6 @@
 {
   "name": "dnd-battle-tracker",
-  "version": "5.117.1",
+  "version": "5.117.2",
   "private": true,
   "dependencies": {
     "@apollo/client": "^3.12.4",

src/components/creature/toolbar/tools/NotesTool.js

@@ -14,10 +14,7 @@ export default function NotesTool({
 }) {
   const [value, setValue] = useState('');
 
-  const notes = allNotes.filter((note) => {
-    const noteRegex = new RegExp(value);
-    return noteRegex.test(note.text);
-  });
+  const notes = allNotes.filter((note) => note.text.toLowerCase().includes(value.toLowerCase()));
 
   const notesDropdownId = `notes-dropdown-${id}`;
   const notesAriaLabel = `${name} notes`;

src/components/view/ViewSwitcher.js

@@ -14,10 +14,11 @@ export default function ViewSwitcher({ views }) {
       <nav className="navigation">
         <ul className="navigation-options">
           {views.map((view, i) => {
+            const { id, title } = view;
             const activeClass = i === viewIndex ? 'navigation-option__active' : '';
             return (
-              <li>
-                <button className={`navigation-option ${activeClass}`} type="button" onClick={() => setViewIndex(i)}>{view.title}</button>
+              <li key={id}>
+                <button className={`navigation-option ${activeClass}`} type="button" onClick={() => setViewIndex(i)}>{title}</button>
               </li>
             );
           })}

test-integration/creature-toolbar/notes-tool.test.js

@@ -139,6 +139,33 @@ describe('Creature note tool', () => {
     dmApp.creatureToolbar.assertCreatureNotesLength('goblin', 1);
   });
 
+  it('ignores case of note when searching notes', async () => {
+    const dmApp = new DmApp();
+    await dmApp.createCreatureForm.addCreature('goblin');
+    await dmApp.creatureToolbar.selectTool('goblin', 'Notes');
+    await dmApp.creatureToolbar.addNote('goblin', 'One');
+    await dmApp.creatureToolbar.typeNote('goblin', 'o');
+    await dmApp.creatureToolbar.assertCreatureNoteExists('goblin', 'One');
+  });
+
+  it('ignores case of search term when searching notes', async () => {
+    const dmApp = new DmApp();
+    await dmApp.createCreatureForm.addCreature('goblin');
+    await dmApp.creatureToolbar.selectTool('goblin', 'Notes');
+    await dmApp.creatureToolbar.addNote('goblin', 'one');
+    await dmApp.creatureToolbar.typeNote('goblin', 'O');
+    await dmApp.creatureToolbar.assertCreatureNoteExists('goblin', 'one');
+  });
+
+  it('allows a note to be searched with special characters', async () => {
+    const dmApp = new DmApp();
+    await dmApp.createCreatureForm.addCreature('goblin');
+    await dmApp.creatureToolbar.selectTool('goblin', 'Notes');
+    await dmApp.creatureToolbar.addNote('goblin', 'one \\');
+    await dmApp.creatureToolbar.typeNote('goblin', '\\');
+    await dmApp.creatureToolbar.assertCreatureNoteExists('goblin', 'one \\');
+  });
+
   it('selects the last note when navigating up from a closed note tool', async () => {
     const dmApp = new DmApp();
     await dmApp.createCreatureForm.addCreature('goblin');