Skip to content

[TCE] Add a first draft of a Persian version #603

Merged
veganstraightedge merged 2 commits intomasterfrom
tce-farsi
Jan 4, 2018
Merged

[TCE] Add a first draft of a Persian version #603
veganstraightedge merged 2 commits intomasterfrom
tce-farsi

Conversation

@astronaut-wannabe
Copy link
Copy Markdown
Contributor

@astronaut-wannabe astronaut-wannabe commented Jan 1, 2018

We wanted to get a first draft of a farsiii version
live on the site ASAP due to some current events.

There probably still needs to be some work on polishing
the CSS and making the site fully RTL (for example, the menu
bar still reads LTR)


demo: https://crimethinc-staging-pr-603.herokuapp.com/tce/فارسی

@astronaut-wannabe astronaut-wannabe self-assigned this Jan 1, 2018
@astronaut-wannabe astronaut-wannabe force-pushed the tce-farsi branch 2 times, most recently from 575a2ce to aa3aa16 Compare January 2, 2018 04:58
@astronaut-wannabe astronaut-wannabe changed the title [WIP] [TCE] Add a farsi tce page [TCE] Add a first draft of a persian version Jan 2, 2018
@astronaut-wannabe
Copy link
Copy Markdown
Contributor Author

astronaut-wannabe commented Jan 2, 2018

@veganstraightedge possible to get a staging deploy on this PR?

I think this is basically a serviceable rough-cut for production.

I will create a PR to track all of the polish that still needs to be done

also, here are the Farsi-specific images: done.zip

@astronaut-wannabe astronaut-wannabe changed the title [TCE] Add a first draft of a persian version [TCE] Add a first draft of a Persian version Jan 2, 2018
@CobyR CobyR temporarily deployed to crimethinc-staging-pr-603 January 2, 2018 22:03 Inactive
@CobyR CobyR temporarily deployed to crimethinc-staging-pr-603 January 2, 2018 22:07 Inactive
We wanted to get a first draft of a farsiii version
live on the site ASAP due to some current events.

There probably still needs to be some work on polishing
the CSS and making the site fully RTL (for example, the menu
bar still reads LTR)
<li><a href="/tce/espanol">Español</a></li>
<li><a href="http://www.crimethinc.com/blog/2016/09/21/to-change-everything-in-11-more-languages/#espanol">Español<br />&nbsp;&nbsp;&nbsp;(América Latina)</a></li>
<li><a href="http://www.crimethinc.com/blog/2016/09/21/to-change-everything-in-11-more-languages/#farsi">فارسی</a></li>
<li><a href="/tce/فارسی">فارسی</a></li>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems to render as valid html, I think it is correct and just looks like this due to the html being ltr and the text being rtl.... not sure though

@veganstraightedge
Copy link
Copy Markdown
Contributor

Images uploaded.

@veganstraightedge
Copy link
Copy Markdown
Contributor

There's a few tweaks that we should do for readability of text on top of images and the size of a couple, but this is pretty dang good enough to merge and launch. I'll make a separate issue for the tweaks.

@veganstraightedge veganstraightedge merged commit 21b5a7b into master Jan 4, 2018
@veganstraightedge veganstraightedge deleted the tce-farsi branch January 4, 2018 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants