snapotter-hq/SnapOtter

Follow-up from #1207 (fixes #1059). The author scoped Escape out to stay under the 400-line limit and offered to take it next; @DevShoaib78, it's yours if you want it. Repro Open /editor, load an image, press L then Shift+L for the Polygonal Lasso. Click three times to drop vertices. The dashed rubber band and the close target on the first vertex appear. Press Escape. Expected: the polygon is discarded, the rubber band and close target disappear, and any existing selection is left alone. Actual: the rubber band and close target stay on screen. The next click adds a fourth vertex, and Enter or a click on the first vertex then closes a polygon the user tried to cancel. In add or subtract mode Escape also wipes the selection the user was building on, because the handler calls setSelection(null) unconditionally. Where it lives apps/web/src/hooks/use-editor-shortcuts.ts:717 is the Escape handler. It clears crop state, selected objects and the committed selection, and has never known abou

What happens today Three keys in the features namespace describe a countdown nothing renders: lessThanMinute: "Less than a minute left", oneMinuteLeft: "~1 minute left", minutesLeft: "~{mins} minutes left", git log -S oneMinuteLeft puts them in d38621d, the original 20-locale commit, and shows no later change. Grepping apps/, packages/ and tests/ for any of the three names returns only the 21 locale files themselves. That is 63 translated strings carried for nothing, and every new locale has to write them. Why nothing caught it The parity machinery is one-directional by design. TranslationKeys and tests/unit/web/i18n-parity.test.ts make sure the 20 locales match en.ts; nothing checks that an en.ts entry is reachable from the app. Gotcha Grep alone does not prove a key unreferenced, for the reason #922 gives: a destructured local (const tt = t.toolSettings[toolId]) hides the property access. It does not apply here, t.features is only ever read through direct property access (erase-objec

What happens today Every locale file in packages/shared/src/i18n/ carries the same line: mfaEnrollmentHeading: "Set up two-factor authentication to continue", All 21 of them, English included. On an instance with a required MFA policy, a Japanese or Arabic admin reaching the enrollment panel on the login page reads that heading in English while every string around it is translated. It is the only string in that state. Comparing every string literal of 26+ characters in each locale file against en.ts turns up 1 to 4 matches per file, and mfaEnrollmentHeading is the only one present in all 20 non-en files. The rest are legitimately identical, e.g. "Original: {width} x {height} px". Why nothing caught it TranslationKeys = DeepStringRecord<typeof en> makes a missing key a tsc error, and tests/unit/web/i18n-parity.test.ts catches extras. Neither looks at values, so a key copied from en.ts and never translated is invisible to both. Where it lives Rendered in apps/web/src/pages/login-page.tsx

From the in-app search box: tambahkan jpg to ico ("add jpg to ico") The capability exists under a name nobody searches for The favicon tool (packages/shared/src/constants.ts:162, "Generate all favicon and app icon sizes") already produces ICO. Someone who wants a single JPG turned into an .ico will not search for "favicon", they will search for "jpg to ico", and that returns nothing. Same shape as #733, where "edit pdf" failed to surface the pdf-edit tool, and the same shape as #1099 for audio. Two ways to fix it, and they are not exclusive Keywords. Add ico and favicon aliases to the favicon tool's keywords so the existing tool is findable by the words people actually type. Cheapest fix, and it helps immediately. A real preset. jpg-to-ico and png-to-ico as BASE_CONFIG entries. ICO is already a supported output format, so this is a settings lock and a narrowed sourceInputs, not new conversion code. Worth it because "one image, one .ico file" is a different job from "generate the full

Two pieces of feedback, both from people who went looking and found nothing: there is no opus to mp3 or to any audio [search] mp3 to ogg The capability is there, the preset isn't .opus and .ogg are both in AUDIO_INPUTS (packages/shared/src/modality.ts), and the generic audio converter handles them. So the first report is wrong on the facts, but right about the experience: searching "opus to mp3" returns nothing, so as far as that user is concerned the feature does not exist. This is the same failure #733 fixed for "edit pdf", where the tool existed and search could not find it. The preset matrix is lopsided Audio presets in packages/shared/src/conversion-presets.ts today: mp4-to-mp3 mov-to-mp3 mkv-to-mp3 webm-to-mp3 avi-to-mp3 mp4-to-wav mov-to-wav mp4-to-ogg m4a-to-mp3 m4a-to-wav aac-to-mp3 aac-to-wav aac-to-flac ogg-to-mp3 ogg-to-wav wav-to-mp3 mp3-to-wav flac-to-mp3 Almost everything converges on mp3 or wav. Going the other way barely exists: mp3-to-w