On-demand language switching with dropdown for translations - #18
Open
sdass1918 wants to merge 44 commits into
Open
On-demand language switching with dropdown for translations#18sdass1918 wants to merge 44 commits into
sdass1918 wants to merge 44 commits into
Conversation
…performance timing
The context-menu click handler must be async because subsequent logic awaits asynchronous checks (e.g. AI availability) before continuing. Without an async listener, await cannot be used and the workflow fails. Co-authored-by: Copilot Autofix powered by AI <[email protected]>
checkAIAvailability() is async, but it's currently used as a boolean without await, so the Promise is always truthy and this guard never blocks when AI is unavailable. Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
…errors Co-authored-by: Copilot Autofix powered by AI <[email protected]>
The parent context menu was registered for all pages, causing it to appear outside web.archive.org playback URLs. Clicking the menu on unsupported pages failed because no content script was injected there ("Receiving end does not exist"). Restrict the menu to Wayback playback URLs to prevent invalid usage.
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
…rformance metrics
…fix screenshot capture timing and append format
…o branching diagram
…ntent for quality analysis The quality analysis feature was broken because append() followed by promptStreaming() does not reliably ground image data with Gemini Nano. The model could not see the screenshot even though it was added to the session history. Fix: pass the image and text prompt together as a single structured message array directly to promptStreaming() instead of using append(). Also removed page content from the quality prompt since it is not needed for quality analysis and only wastes context window space. The content extraction step is also skipped for quality actions to save time.
sdass1918
changed the title
… listeners and quality panel ordering
sdass1918
force-pushed
the
feature/language-switching
branch
from
July 6, 2026 14:38
510b12a to
cecb2e6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey team,
This PR adds an on-demand language switching feature — instead of being locked to the language you selected before running the analysis, you can now switch languages anytime from a dropdown.
What changed:
All 4 files touched: background.js, content/content.js, content/ui/overlay.js, content/ui/styles.js