jelmervdl / translatelocally-web-ext

TranslateLocally for the Browser is a web-extension that enables client side in-page translations for web browsers.
https://addons.mozilla.org/en-GB/firefox/addon/translatelocally-for-firefox/
Mozilla Public License 2.0
65 stars 3 forks source link

translation popup should either close or show success once translation is complete #54

Closed cadeyrn closed 1 year ago

cadeyrn commented 1 year ago

Right now the translation popup keeps open once the translations is complete and still shows the progress bar and the "stop translating" button. As the button can be used to show the original text it makes sense to still show the button. But the button label "stop translating" in combination with the fact that nothing else changes in the popup can be a bit misleading as it feels as the translation is still in progress.

So my proposal is to either close the popup or to do something in the panel to indicidate that the translation progress is complete. One idea would be to change the button label to something like "show original text".

jelmervdl commented 1 year ago

This is a good point. Technically it is this way because it is in a "state of translation" where any updates to the page will be translated as they occur. But from a user's point of view, if there's no translations pending, it should show as done.

I also agree that "Stop Translating" is not accurate anymore. "Show original" is a better label for the button because regardless of how far the page has progressed translating, that's what the effect of that button is.

There is a brief moment between you asking for a translation and the translation starting where the model is downloaded etc. There the text "Stop translating" makes more sense since the entire page is still the original page.

I'm going to change it to: