search

electron / website

:electron: The Electron website
https://electronjs.org
Apache License 2.0
113 stars 125 forks source link

Translatable link missing in https://www.electronjs.org/docs/latest/api/browser-window #357

Open albu77 opened 1 year ago

albu77 commented 1 year ago

In the api/browser-window the link shown on the following picture is pointing to a valid anchor but as there is no hidden text corresponding to this anchor link in the crowdin system the link is not working in the locale version.

Capture

erickzhao commented 1 year ago

@albu77 Does it not work if you change the anchor to #d%C3%A9finition-de-la-propri%C3%A9t%C3%A9-backgroundcolor?

erickzhao commented 1 year ago

I tried fixing it here: https://crowdin.com/translate/electron/258633/en-fr#7514508

albu77 commented 1 year ago

Good job, it's working. But to my sense, generally speaking it would be better if we could do in the same manner (the right one for me) than in/docs/latest/tutorial/security. as it 's shown below: Capture And speaking about this one the -7 option (issue #264) is still unsolved. All this stuff around links is a little bit puzzling for me because I have not a general overview of the interaction of docs in the code, docs in the "website", docusaurus etc...

erickzhao commented 1 year ago

@albu77 I'm unfamiliar with this part of the translation flow for Crowdin so let me figure out what the best practice for this will be.

albu77 commented 1 year ago

Yes I think it's interesting if you can figure it because it's a case often met