Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

revert "Flip RTL text before sending it #184" #1024

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

amiremohamadi
Copy link
Contributor

@amiremohamadi amiremohamadi commented Oct 13, 2024

seems like it's been fixed in kodi while ago and we should not reverse the input string anymore.
@SyncedSynapse

@basilgello
Copy link

If minimal version of Kodi that Kore supports has the original issue fixed, then simply removing the code is enough. Otherwise, you need to guard that with a flag that is set once on connection estabilishment.

@amiremohamadi
Copy link
Contributor Author

amiremohamadi commented Oct 14, 2024

@basilgello reversing the text has never been a correct solution. imagine a situation in which our text is a mixture of RTL and LTR words. therefore, I believe it's a correct behavior not to handle it on the Kore.

@basilgello
Copy link

OK, just checking as I was not aware of the problem (and its "fix")

@SyncedSynapse
Copy link
Member

SyncedSynapse commented Oct 14, 2024

No idea about what the correct behaviour should be.
@amiremohamadi Do you have a link to the Kodi PR that fixed the issue?
@royiarchy Do you have any input into this.

If the issue was fixed in a specific version of Kodi, what basilgello was referring to was keeping the code, but guarding it with something like

if (! hostManager.getHostInfo().isLeiaOrLater) {
    // Removed code to reverse text
}

Not sure if necessary

@amiremohamadi
Copy link
Contributor Author

@SyncedSynapse I guess it's solved in this commit.
IMO keeping the previous code (reversing the whole text) for the older versions is unnecessary as it will still mess up mixed RTL/LTR texts (even in the older versions of kodi).

@SyncedSynapse SyncedSynapse merged commit a891d5f into xbmc:master Oct 16, 2024
1 check passed
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