Skip to content

Conversation

@Zahra-khanzadeh
Copy link
Contributor

Hi @mmcky
I've updated the about_py.md and python_by_example.md lectures:
-Updated translated text to match the latest English versions
-Adjusted image folders and paths accordingly
-Adjusted the placement of parentheses in code examples included in the explanation (in RTL layout, parentheses were displayed in reverse order, for example ()append instead of append(), It might seem a bit odd, but in Persian, when we read technical text and encounter an English expression, we read it left to right even in the middle of a Persian paragraph :) , so I manually flipped them in the Markdown to ensure correct visual alignment on the website.)

Thanks!

@netlify
Copy link

netlify bot commented Oct 28, 2025

Deploy Preview for majestic-griffin-10b166 ready!

Name Link
🔨 Latest commit 7509f84
🔍 Latest deploy log https://app.netlify.com/projects/majestic-griffin-10b166/deploys/690083ec559ade0008a9d22f
😎 Deploy Preview https://deploy-preview-14--majestic-griffin-10b166.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link

@github-actions github-actions bot temporarily deployed to pull request October 28, 2025 08:56 Inactive
@mmcky
Copy link
Contributor

mmcky commented Oct 29, 2025

Thanks @Zahra-khanzadeh.

-Adjusted the placement of parentheses in code examples included in the explanation (in RTL layout, parentheses were displayed in reverse order, for example ()append instead of append(), It might seem a bit odd, but in Persian, when we read technical text and encounter an English expression, we read it left to right even in the middle of a Persian paragraph :) , so I manually flipped them in the Markdown to ensure correct visual alignment on the website.)

This is an interesting problem. I'll have a think about how we might be able to fix this automatically. I can't think of an easy way just yet. Is () the only case? We could add a special post-processor to adjust them in the translation process.

@Zahra-khanzadeh
Copy link
Contributor Author

Thanks a lot, @mmcky .I really appreciate your time and effort on checking the parentheses issue.

Is () the only case?

Actually, there are a few other small issues I've noticed as well.

  1. In the last hint of the python_by_example.md lecture, the final line $area = \pi * radius^2$ breaks the text incorrectly when translated to Persian ( for example, it shows as $مساحت = \pi * شعاع^2$ ).
    As shown in the attached image, the math formula get split from left to right. (I tried several ways to fix it, but none worked so far.)
Screenshot 2025-10-29 130025
  1. In the python_by_example.md, lines 348-352 should ideally be displayed left to right, since they contain code examples. Otherwise, the text alignment becomes confusing, as shown in the attached screenshot.
Screenshot 2025-10-28 105720
  1. Finally, it would be great if the titles for Exercises, Hints, and Solutions could also be translated into Persian in the RTL layout. I've attached a screenshot for reference.
Screenshot 2025-10-29 130334

Thanks again, Matt. The RTL layout is improving so much with each update, and I really appreciate all the work you're putting into it.

@mmcky
Copy link
Contributor

mmcky commented Oct 30, 2025

great feedback - thanks @Zahra-khanzadeh

@mmcky
Copy link
Contributor

mmcky commented Oct 30, 2025

I'll merge this PR just to keep the main branch up to date with the latest.

@mmcky mmcky merged commit 363fe6f into main Oct 30, 2025
6 checks passed
@mmcky mmcky deleted the update-aboutpy-and-pybyex branch October 30, 2025 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants