Use ul for search results

roman-r-m · Feb 7, 2021 · bcbb15b · bcbb15b
1 parent fee1127
commit bcbb15b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/plugin/index.ts b/src/plugin/index.ts
@@ -64,8 +64,8 @@ joplin.plugins.register({
 
 		joplin.views.dialogs.setHtml(resourceSearch, `
 		<div id="resource-search">
-			<input id="query-input" type="text">
-			<div id="search-results"></div>
+			<input id="query-input" type="text" autofocus>
+			<ul id="search-results"></ul>
 		</div>
 		`);
 
@@ -77,7 +77,7 @@ joplin.plugins.register({
 					console.log(`results: ${JSON.stringify(result)}`);
 					return await transformResult(result);
 				case 'goto':
-					// TODO scroll to resource
+					// TODO scroll to the resource position within the note
 					await joplin.views.panels.hide(resourceSearch);
 					await joplin.commands.execute('openNote', msg.noteId);
 					break;

diff --git a/src/webview/index.ts b/src/webview/index.ts
@@ -82,7 +82,7 @@ class SearchDialog {
             this.selectedIndex = -1;
             for (let i = 0; i < this.results.length; i++) {
                 const searchResult = this.results[i];
-                const row = document.createElement('div');
+                const row = document.createElement('li');
                 row.setAttribute('class', 'search-result-row');
                 searchResults.appendChild(row);