Skip to content

Conversation

@ofekashery
Copy link
Contributor

@ofekashery ofekashery commented Nov 24, 2020

The beginning of Hebrew support 🥳

@AustinShalit
Copy link
Member

This looks great! If it helps you at all (not sure if it will) take a look at the PR here: readthedocs/sphinx_rtd_theme#986

@Daltz333
Copy link
Member

A very pleasant surprise!

@ofekashery
Copy link
Contributor Author

Thanks! I think it's ready.

By the way, it seems like something is broken with the Hebrew docs in production - it works for me locally.

@Daltz333
Copy link
Member

Hebrew isn't enabled 😄

@Daltz333
Copy link
Member

Something that would be nice, if things like admonitions (warnings/notes), hoverxref (hover tooltip content), and the extra things we have in the docs (code-tabs) could be RTL aligned.

@ofekashery
Copy link
Contributor Author

The admonitions and code-tabs are already RTL aligned.
Do you have an example of a page that uses the hoverxref? I didn't find it.

image
image

@Daltz333
Copy link
Member

@Daltz333 Daltz333 added Type: Style/CSS Adjusts styling of the website. Waiting On: FIRST Waiting on something from FIRST labels Nov 25, 2020
@Daltz333
Copy link
Member

@Kevin-OConnor what are your thoughts on the CSS styling and changes

@ofekashery
Copy link
Contributor Author

Here is how it looks:
image

@AustinShalit AustinShalit removed the Waiting On: FIRST Waiting on something from FIRST label Nov 27, 2020
@AustinShalit
Copy link
Member

I am not sure why this would look this way but does the Zayin look backward to you?

image

@ofekashery
Copy link
Contributor Author

Actually, if I focus on that, it looks a bit backward, but it's part of Heebo's typography (We use Heebo as our primary font at FIRST Israel).

@AustinShalit
Copy link
Member

Ah ok. I don't want to go messing with the typography.

Someone pointed out to me that the C++ gets converted to ++C in the image you posted. I think this is not standard?

@ofekashery
Copy link
Contributor Author

ofekashery commented Nov 27, 2020

It's not expected but unfortunately we don't have much to do with it.
It's generally caused because the characters being written right-to-left, and the ++ are locale-free characters, so they are rendered after the C. If there was another English char after the ++ it would be okay.

A possible solution is to wrap it in a span with left-to-right direction (<span dir="ltr">C++</span>), but I'm not sure how to do that on the translations side.

This issue seems to also exist in other sites that write C++ in a Hebrew sentence...

@Daltz333
Copy link
Member

Daltz333 commented Dec 2, 2020

Ah. I'm gonna spend a bit of time testing locally tomorrow, so expect a review then

Co-authored-by: Austin Shalit <[email protected]>
@Daltz333 Daltz333 merged commit e0476b8 into wpilibsuite:master Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Style/CSS Adjusts styling of the website.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants