Add the ability to auto close the tab on approval

[senxd]

When approving/sending feedback, the default behavior is to show a full-page indicator. I have added an option in the settings to close the tab after submitting instead.

[flex-yj-kim]

Looks great! Could you share some screenshots of this feature? It would be much easier to understand.

[senxd]

recording.mov

Here is a video. By default, it is toggled off, and nothing is changed. When enabled, it closes the tab once you approve/send feedback. It just helps with tab clutter as I close it regardless.

[backnotprop]

Big feature. Taking a look.

I just got back from my only vacation this year and will get these PRs going.

[backnotprop]

Code review

Found 1 issue:

1. window.close() will be blocked by browser security in most cases. Plannotator opens tabs via system commands (open, xdg-open, start), not window.open(), so browsers block the close attempt. All 4 call sites have no fallback UI or indication to the user when the close fails -- the tab just stays open silently. This limitation was previously documented in Issue #41. Consider adding a fallback (e.g., showing the existing success overlay when window.close() doesn't work).

plannotator/packages/editor/App.tsx

Lines 546 to 550 in 2d7b035

	});
	if (getAutoClose()) {
	window.close();
	}
	setSubmitted('approved');

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[backnotprop]

Ignore that review. I'm not seeing an issue with newer browsers.