Set layout direction based on locale

Signed-off-by: ali ghorbani <[email protected]>
nextcloud · Aug 20, 2024 · a6268fb · a6268fb
1 parent 560282a
commit a6268fb
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 2 deletions.
diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
@@ -49,7 +49,7 @@
 		<?php emit_script_loading_tags($_); ?>
 		<?php print_unescaped($_['headers']); ?>
 	</head>
-	<body id="<?php p($_['bodyid']);?>" <?php foreach ($_['enabledThemes'] as $themeId) {
+	<body dir="<?php p($_['direction']); ?>" id="<?php p($_['bodyid']);?>" <?php foreach ($_['enabledThemes'] as $themeId) {
 		p("data-theme-$themeId ");
 	}?> data-themes=<?php p(join(',', $_['enabledThemes'])) ?>>
 		<?php include 'layout.noscript.warning.php'; ?>

diff --git a/lib/private/L10N/Factory.php b/lib/private/L10N/Factory.php
@@ -58,6 +58,12 @@ class Factory implements IFactory {
 		'pt_BR', 'pt_PT', 'da', 'fi_FI', 'nb_NO', 'sv', 'tr', 'zh_CN', 'ko'
 	];
 
+	const RTL_LANGUAGES = [
+		'ae', 'ar', 'arc', 'arz', 'bcc', 'bqi', 'ckb', 'dv', 'fa', 'glk',
+		'ha', 'he', 'khw', 'ks', 'ku', 'mzn', 'nqo', 'pnb', 'ps', 'sd', 'ug',
+		'ur', 'uzs', 'yi',
+	];
+
 	private ICache $cache;
 
 	public function __construct(
@@ -364,6 +370,15 @@ public function languageExists($app, $lang) {
 		return in_array($lang, $languages);
 	}
 
+	public function getLanguageDirectionFromLocale($locale): string
+	{
+		if (in_array($locale, self::RTL_LANGUAGES)) {
+			return "rtl";
+		}
+
+		return "ltr";
+	}
+
 	public function getLanguageIterator(?IUser $user = null): ILanguageIterator {
 		$user = $user ?? $this->userSession->getUser();
 		if ($user === null) {

diff --git a/lib/private/TemplateLayout.php b/lib/private/TemplateLayout.php
@@ -191,13 +191,15 @@ public function __construct($renderAs, $appId = '') {
 		} else {
 			parent::__construct('core', 'layout.base');
 		}
-		// Send the language and the locale to our layouts
+		// Send the language, locale, and direction to our layouts
 		$lang = \OC::$server->get(IFactory::class)->findLanguage();
 		$locale = \OC::$server->get(IFactory::class)->findLocale($lang);
+		$direction = \OC::$server->getL10NFactory()->getLanguageDirectionFromLocale($locale);
 
 		$lang = str_replace('_', '-', $lang);
 		$this->assign('language', $lang);
 		$this->assign('locale', $locale);
+		$this->assign('direction', $direction);
 
 		if (\OC::$server->getSystemConfig()->getValue('installed', false)) {
 			if (empty(self::$versionHash)) {

diff --git a/lib/public/L10N/IFactory.php b/lib/public/L10N/IFactory.php
@@ -103,6 +103,14 @@ public function languageExists($app, $lang);
 	 */
 	public function localeExists($locale);
 
+	/**
+	 * Return the current language direction from locale
+	 *
+	 * @param string $locale
+	 * @return string
+	 */
+	public function getLanguageDirectionFromLocale($locale): string;
+
 	/**
 	 * iterate through language settings (if provided) in this order:
 	 * 1. returns the forced language or: