website icon indicating copy to clipboard operation
website copied to clipboard

Changing page language shows modal in current language

Open davenewham opened this issue 3 years ago • 16 comments

i.e if on the german website and I try to change the language, I will be shown a modal in German warning me image.

I think it would be more expected to have both target selected language appear, and current page language. Perhaps the buttons should be in the target language?

davenewham avatar Sep 24 '21 07:09 davenewham

Can I help out with this issue?

TomerPacific avatar Oct 03 '21 08:10 TomerPacific

Hi @TomerPacific, I've been working on some changes to this I18NWidget and suggesting other languages on the PR #615. This PR has now changed the behaviour, to where the target language is shown in the modal (instead of the current language). If you wanted to help out, you could try and figure out how to display both current and target language? Maybe with a HR element between them?

davenewham avatar Oct 03 '21 09:10 davenewham

@davenewham , I've looked over the PR. Would it be best that I branch out of the PR's branch and opened a new PR for that?

TomerPacific avatar Oct 03 '21 10:10 TomerPacific

Yes, I think so. Thanks for helping 😊

davenewham avatar Oct 03 '21 10:10 davenewham

The PR is now merged, so you can also work straight from master if you haven't started already. :)

baltpeter avatar Oct 03 '21 11:10 baltpeter

@baltpeter, should I open a new issue then or would you like to keep this one open?

TomerPacific avatar Oct 03 '21 18:10 TomerPacific

@TomerPacific I see no reason to open a new issue, let's use this one. I can assign you if you want.

baltpeter avatar Oct 04 '21 09:10 baltpeter

@baltpeter, sure let's do that. Just wanted to make sure I am aligned with your working style.

TomerPacific avatar Oct 04 '21 09:10 TomerPacific

@baltpeter, just to make sure I understand what needs to be done, you would like to display in the modal, the current language of the website and also the target language selected?

If this is so, would you like that the different versions of the text appear inside the buttons or somewhere else? I.E. at the top of the modal? website

TomerPacific avatar Oct 04 '21 12:10 TomerPacific

just to make sure I understand what needs to be done, you would like to display in the modal, the current language of the website and also the target language selected?

Yes, exactly. Otherwise, a user who accidentally selects the wrong language might not find their way back.

If this is so, would you like that the different versions of the text appear inside the buttons or somewhere else? I.E. at the top of the modal?

Not sure. As @davenewham said, we haven't figured out a design for that yet. That's also part of the scope of this issue.

baltpeter avatar Oct 04 '21 14:10 baltpeter

Maybe hide the other language in a <detail> element?

zner0L avatar Oct 06 '21 14:10 zner0L

Which one is the other? Both are equally important imo.

mal-tee avatar Oct 06 '21 14:10 mal-tee

I think the visible text should be in the language the user is trying to change to, while the text in the language the website is currently in should be hidden in a <detail>. Also, I think the button for "Stay here" should definitely be in the current language.

zner0L avatar Oct 07 '21 19:10 zner0L

@baltpeter, just wanted to update that I am having some issues with setting up the local environment. I am working on a Windows machine and am following the development guide. It seems that running the deploy.sh script takes a really long time (it gets stuck on the phase of Renaming JSON Files). Don't know if that is the expected behavior.

TomerPacific avatar Oct 12 '21 18:10 TomerPacific

As far as I know, it's unfortunately quite common for this to take a while on Windows. We're working on improving that a bit at least in #674. But for Windows-related questions, you should definitely ask @mal-tee. I don't have any experience with that.

baltpeter avatar Oct 13 '21 09:10 baltpeter

@baltpeter - might be worthwhile to assign someone else this issue. I do not know if I will be able to overcome to specific platform issues and I do not want to waste your time.

TomerPacific avatar Oct 17 '21 16:10 TomerPacific