Bug fixes and UX improvements

- Fix hover preview race condition during drag (retry dismiss on mousemove)
- Fix drag-then-navigate bug (any pixel movement suppresses navigation)
- Fix no-pin-no-icon state: preview shows placeholder, save blocked
- Fix unused parameter in onMouseUp
- Measurement text renders above markers (SVG overlay z-index)
- Modal titles: "Create marker/template/layer" for new items
- Layer edit modal uses title bar
- Build runs lint before compile
- Bump version to 1.1.2

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: matthttam

.github/workflows/release.yml

@@ -3,7 +3,7 @@ name: Release Obsidian Plugin
 on:
   push:
     tags:
-      - "[0-9]+.[0-9]+.[0-9]+"
+      - '[0-9]+.[0-9]+.[0-9]+'
 
 permissions:
   contents: write
@@ -16,7 +16,7 @@ jobs:
 
       - uses: actions/setup-node@v6
         with:
-          node-version-file: ".nvmrc"
+          node-version-file: '.nvmrc'
 
       - name: Install dependencies
         run: npm install

CLAUDE.md

@@ -57,32 +57,41 @@ Tests live in `tests/` (outside `src/` so the Obsidian review bot doesn't scan t
 This plugin is published to the Obsidian community directory. All code is scanned by an automated review bot using `eslint-plugin-obsidianmd` and `@typescript-eslint`. Run `npx eslint src/` locally before committing to catch issues early.
 
 ### UI text must use sentence case
+
 All `.setName()`, `.setButtonText()`, `.setPlaceholder()`, `.setDesc()`, and heading text must use sentence case (first word capitalized, rest lowercase). Exceptions: proper nouns (TTRPG), acronyms (ID, SVG). Example: "Default marker scale", not "Default Marker Scale".
 
 ### No inline styles
+
 Never use `element.style.X = ...` directly. Use CSS classes instead:
+
 - Show/hide: `el.addClass("ttrpgmap-hidden")` / `el.removeClass("ttrpgmap-hidden")` (defined in styles.css)
 - Cursors: `ttrpgmap-cursor-grab`, `ttrpgmap-cursor-crosshair`, `ttrpgmap-cursor-copy`
 - Image rendering: `ttrpgmap-pixelated`
 - For truly dynamic values (positions, transforms, CSS custom properties), use `// eslint-disable-next-line obsidianmd/no-static-styles-assignment`
 
 ### Headings use the Setting API
+
 Use `new Setting(containerEl).setName("...").setHeading()` instead of `createEl("h2")` or `createEl("h3")`.
 
 ### No innerHTML writes
+
 Use `parent.empty()` to clear content. Use `DOMParser` + `appendChild` for injecting SVG strings. Reading innerHTML in tests is OK.
 
 ### No browser confirm()
+
 Use an Obsidian `Modal` with buttons that resolve a Promise instead of `confirm()`.
 
 ### No async lifecycle overrides
+
 Don't declare `onload()`, `onOpen()`, etc. as `async`. Wrap the async body in `void (async () => { ... })()` inside a synchronous method.
 
 ### Promise handling
+
 - Don't pass `async` callbacks to `.onChange()`, `.onClick()`, or `addEventListener`. Either remove async and prefix promise calls with `void`, or wrap in `void (async () => { ... })()`.
 - Prefix fire-and-forget promise calls with `void` (e.g., `void plugin.dataManager.saveSettings(...)`).
 
 ### No explicit `any` in production code
+
 Use proper types. For undocumented Obsidian APIs (e.g., `setSubmenu()`, `app.setting`), use `// eslint-disable-next-line @typescript-eslint/no-explicit-any`. In test/mock files, eslint-disable comments are acceptable for mock objects.
 
 ## Key conventions

README.md

@@ -50,6 +50,7 @@ Add an empty `ttrpgmap` code block to any note:
 
 ````markdown
 ```ttrpgmap
+
 ```
 ````
 
@@ -105,7 +106,6 @@ Configure rounding in the measurement panel to snap distances to the nearest mul
 <img width="803" height="624" alt="image" src="https://github.com/user-attachments/assets/5cfcdd20-8d77-4bbe-a943-4fe0faca4f65" />
 <img width="800" height="793" alt="image" src="https://github.com/user-attachments/assets/cd25067e-1f9f-4c8d-8d4f-55f31f966307" />
 
-
 ### 5. Manage templates
 
 Open **Settings** > **TTRPG Maps** to create and organize marker templates. Templates define default values for color, icon, shape, direction, and more. Use **Save & Update Markers** to push template changes to all existing markers that use it.
@@ -114,11 +114,11 @@ Open **Settings** > **TTRPG Maps** to create and organize marker templates. Temp
 
 ## How data is stored
 
-| Data | Location | Managed by |
-|---|---|---|
-| Map config (image, zoom, dimensions) | Code block in your markdown file | Plugin writes back on settings change |
-| Markers, scale, layers, rounding | `.ttrpgmap/{mapId}.json` sidecar files | Plugin (debounced 300ms saves) |
-| Global settings and templates | `.obsidian/plugins/ttrpg-maps/data.json` | Obsidian `loadData`/`saveData` |
+| Data                                 | Location                                 | Managed by                            |
+| ------------------------------------ | ---------------------------------------- | ------------------------------------- |
+| Map config (image, zoom, dimensions) | Code block in your markdown file         | Plugin writes back on settings change |
+| Markers, scale, layers, rounding     | `.ttrpgmap/{mapId}.json` sidecar files   | Plugin (debounced 300ms saves)        |
+| Global settings and templates        | `.obsidian/plugins/ttrpg-maps/data.json` | Obsidian `loadData`/`saveData`        |
 
 Sidecar files keep marker data separate from note content. You can safely commit them to version control or sync them across devices.