Skip to content

Fix formatting in Arabic index.html file#3678

Closed
rania33840-netizen wants to merge 1 commit into
github:mainfrom
rania33840-netizen:patch-1
Closed

Fix formatting in Arabic index.html file#3678
rania33840-netizen wants to merge 1 commit into
github:mainfrom
rania33840-netizen:patch-1

Conversation

@rania33840-netizen
Copy link
Copy Markdown

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


@rania33840-netizen rania33840-netizen requested a review from a team as a code owner May 26, 2026 12:25
Copy link
Copy Markdown
Contributor

@kenyonj kenyonj left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @rania33840-netizen! I want to make sure I am not misreading the intent here. Looking at the diff:

---
layout: index
lang: ar
title: إرشادات المشاريع مفتوحة المصدر
permalink: /ar/
+------
+layout: index
+lang: ar
+title: إرشادات المشاريع مفتوحة المصدر
+permalink: /ar/
---

This duplicates the entire YAML frontmatter block inside the existing one, and separates the two with ------ (six dashes) rather than --- (three dashes). Jekyll's frontmatter parser expects exactly --- as the opening and closing delimiters, so the six-dash line is not a valid separator. The result would be invalid frontmatter and the Arabic landing page might not render correctly.

The original file on main already matches every other locale's index.html structure (compare for instance _articles/ru/index.html, _articles/pl/index.html, _articles/de/index.html):

---
layout: index
lang: ar
title: إرشادات المشاريع مفتوحة المصدر
permalink: /ar/
---

Was there a specific rendering issue you were trying to fix? If you can describe what you were seeing (e.g. broken layout, wrong title, RTL display issue), I am happy to help figure out the right fix. Otherwise this PR can probably be closed.

Heads-up: I am not a native Arabic speaker, so I focused on the structural side of the file rather than the Arabic content itself.

@kenyonj
Copy link
Copy Markdown
Contributor

kenyonj commented Jun 1, 2026

going to close this out since it seems to be misformated and unnecessary, please reopen with proper changes if needed

@kenyonj kenyonj closed this Jun 1, 2026
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.

2 participants