vitepress icon indicating copy to clipboard operation
vitepress copied to clipboard

docs: add Persian language

Open amir78729 opened this issue 1 year ago • 7 comments
trafficstars

Description

This PR adds the Persian language locale to VItePress Doc

Linked Issues

fixes #4047

Additional Context


[!TIP] The author of this PR can publish a preview release by commenting /publish below.

amir78729 avatar Jul 30 '24 17:07 amir78729

Please update lunaria.json too. And I need someone from the community to review the translations. I also need feedback on RTL UI. Let me know which things look weird and how they should look, I will adjust the styles.

brc-dd avatar Aug 03 '24 14:08 brc-dd

@brc-dd maybe this can help you (or not ;) ): https://github.com/elk-zone/elk/blob/main/CONTRIBUTING.md#rtl-support

userquin avatar Aug 03 '24 14:08 userquin

Please update lunaria.json too. And I need someone from the community to review the translations. I also need feedback on RTL UI. Let me know which things look weird and how they should look, I will adjust the styles.

@brc-dd Thanks, sure.

  • Nav: spacings between items are not correct

image

  • Hero: text-align should be right

image

  • Sidebar: it's better to put the sidebar on the right side of the page

image

  • Search input: spacings are not correct

image

  • Search modal: spacings in the footer and inside the input (the space between the icon and the text)

image

  • Change language select: when using RTL, the spacings of the Persian options are not correct:

image

amir78729 avatar Aug 06 '24 13:08 amir78729

@brc-dd Are there any updates on fixing RTL issues? Is there anything I can help with?

amir78729 avatar Aug 18 '24 07:08 amir78729

Ah, sorry, no. I'm bit busy recently with day job. Will try to look into by next week.

brc-dd avatar Aug 18 '24 09:08 brc-dd

Ah, sorry, no. I'm bit busy recently with day job. Will try to look into by next week.

No problem. Let me know if there's anything I can help you with.

amir78729 avatar Aug 26 '24 15:08 amir78729

Any updates? @brc-dd

amir78729 avatar Nov 30 '24 09:11 amir78729

Sorry for the delay.

Can you check the preview again and list out the issues?

Also, the individual headings need anchors, so that switching locales can keep the position same. See https://github.com/vuejs/vitepress/blob/main/docs/ru/guide/asset-handling.md?plain=1 for example (notice things inside {...})

brc-dd avatar Nov 30 '24 09:11 brc-dd

Sorry for the delay.

Can you check the preview again and list out the issues?

Also, the individual headings need anchors, so that switching locales can keep the position same. See https://github.com/vuejs/vitepress/blob/main/docs/ru/guide/asset-handling.md?plain=1 for example (notice things inside {...})

Everything looks fine.

There's another thing that I suggest changing. If the text color has opacity in languages like Persian (where the characters are connected), the overlap of characters will be bolder. Can we remove the opacity and use solid text colors to avoid the bolder overlap of characters?

image

amir78729 avatar Dec 08 '24 10:12 amir78729

Updated the colors. Please add the headers thing I mentioned earlier and this will be good to merge I guess 👀

brc-dd avatar Dec 09 '24 14:12 brc-dd

external link icons shouldn't be changed with transform: rotate(-90deg)?

imagen

This should be the result:

imagen

userquin avatar Dec 11 '24 15:12 userquin

updated

brc-dd avatar Dec 11 '24 16:12 brc-dd