[tx-trac] #451: RTL issue in Transifix
Transifex Trac
--------------------+-------------------------------------------------------
Reporter: lashar | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Core | Version:
Keywords: | Easy_task: 0
Review: | Blockedby:
Blocking: | Needs_patch: 1
--------------------+-------------------------------------------------------
Hey.
As far as I remember Persian (fa) and Balochi are ther first languages
that are available in Tx which are RTL langugages. As I check
screenshot(http://img697.yfrog.com/img697/2467/mostafa.png) provided by
OgMaciel I see that text decoration in django for showing text in fa
locale in Right rather than left.
--
Ticket URL: <http://transifex.org/ticket/451>
Transifex <http://transifex.org/>
Integrated translation platform
Transifex Trac
--------------------+-------------------------------------------------------
Reporter: lashar | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Core | Version:
Keywords: | Easy_task: 0
Review: | Blockedby:
Blocking: | Needs_patch: 1
--------------------+-------------------------------------------------------
Old description:
> Hey.
> As far as I remember Persian (fa) and Balochi are ther first languages
> that are available in Tx which are RTL langugages. As I check
> screenshot(http://img697.yfrog.com/img697/2467/mostafa.png) provided by
> OgMaciel I see that text decoration in django for showing text in fa
> locale in Right rather than left.
Hey.
As far as I remember Persian (fa) and Balochi are ther first languages
that are available in Tx which are RTL langugages. As I check
screenshot(http://img697.yfrog.com/img697/2467/mostafa.png) provided by
OgMaciel I see that text decoration in django for showing text in fa
--
--
Ticket URL: <http://transifex.org/ticket/451#comment:1>
Transifex Trac
--------------------+-------------------------------------------------------
Reporter: lashar | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Core | Version:
Keywords: | Easy_task: 0
Review: | Blockedby:
Blocking: | Needs_patch: 1
--------------------+-------------------------------------------------------
Comment(by diegobz):
We need to override the css style when the LANGUAGE_BIDI is true
Example:
{{{
{% get_current_language_bidi as LANGUAGE_BIDI %}
{% if LANGUAGE_BIDI %}
<link media="screen" href="/site_media/css/base-bidi.css"
type="text/css" rel="stylesheet" />
{% endif %}
}}}
References:
http://whatsup.org.il/meir/talks/django-i18n/django-i18n.html
http://www.djangobook.com/en/1.0/chapter18/
--
Ticket URL: <http://transifex.org/ticket/451#comment:2>
Transifex Trac
Type: defect | Status: new
Priority: major | Milestone:
Component: Core | Version:
Blockedby: | Blocking:
Ready_for_review: 0 | Review:
Needs_patch: 1 |
---------------------------+------------------------------------------------
Changes (by lashar):
* keywords: => rtl
* ready_for_review: => 0
Comment:
does the new release contain this patch? any commit?
--
Ticket URL: <http://transifex.org/ticket/451#comment:3>
Transifex Trac
Reporter: lashar | Owner: diegobz
Type: defect | Status: assigned
Priority: major | Milestone: 0.8.1
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
Changes (by glezos):
* status: new => assigned
* ready_for_review: 0 => 1
* easy_task: 0 => 1
* needs_patch: 1 => 0
* component: Core => View/Templates
* milestone: => 0.8.1
* owner: => diegobz
Comment:
lashar, oh sorry about this. The ticket wasn't attached to the release and
kinda slipped under our feet.
Do we have a way to test this patch? Is it working against the current
mainline HEAD/tip?
Also, could you please include some screenshots and/or ways to test it?
I'm attaching it to 0.8.1, so if you can provide us with the above we
could ship it very soon.
--
Ticket URL: <http://transifex.org/ticket/451#comment:4>
Transifex Trac
Reporter: lashar | Owner: diegobz
Type: defect | Status: assigned
Priority: major | Milestone: 0.8.1
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
[http://rapidshare.in.ua/ новые картинки]
--
Ticket URL: <http://transifex.org/ticket/451#comment:5>
--
You received this message because you are subscribed to the Google Groups "transifex-updates" group.
To post to this group, send email to transife...@googlegroups.com.
To unsubscribe from this group, send email to transifex-upda...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/transifex-updates?hl=en.
Transifex Trac
Reporter: lashar | Owner: diegobz
Type: defect | Status: assigned
Priority: major | Milestone: 0.8.1
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
Comment(by Yaron):
Your memory is not so good after all, Hebrew and Arabic are here as
well...
Is there a target version for this fix?
--
Ticket URL: <http://transifex.org/ticket/451#comment:5>
Transifex Trac
Reporter: lashar | Owner: diegobz
Type: defect | Status: assigned
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
* milestone: 0.8.1 => 1.0
Comment:
Yaron, can we have some screenshots showing the problem? We have a patch,
BUT we need to know if it actually fixes the issue.
--
Ticket URL: <http://transifex.org/ticket/451#comment:6>
Transifex Trac
Reporter: lashar | Owner: diegobz
Type: defect | Status: assigned
Priority: major | Milestone: 1.0
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
Comment(by Yaron):
[[Image(http://www.mediafire.com/imgbnc.php/ff09f3bf551cd7077219cd60207cbdf25g.jpg)]]
OK, I'll describe what I did since the patch is a little different:
I moved the Red\Green\Blue bar on the left to the right (border-right
instaed of border left) and also added "direction: rtl" to the same class.
This should be done to all RTL languages such as:
Hebrew
Arabic
Persian
Kashmiri
Beluchi
Divehi
Yiddish
Syriac
There are several more but since some of them has several scripts I cannot
guarantee that this patch is usefull to them (maybe the should have the
ability to choose).
Kind regards,
Yaron Shahrabani.
--
Ticket URL: <http://transifex.org/ticket/451#comment:7>
Transifex Trac
Reporter: lashar | Owner: alup
Type: defect | Status: assigned
Priority: major | Milestone: 1.0.1
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
* owner: diegobz => alup
* milestone: 1.0 => 1.0.1
Comment:
Pushing to [milestone:1.0.1] until we have a better patch.
--
Ticket URL: <http://transifex.org/ticket/451#comment:8>
Transifex Trac
Reporter: lashar | Owner: alup
Type: defect | Status: assigned
Priority: major | Milestone: 1.0.1
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
Comment(by Yaron):
This patch is different than the one that we need to display the entire
website as RTL so there are 2 patches:
One that enables better usage of the English Transifex system while
translating RTL languages (this patch) while the other patch needed is a
full RTLing of the system which is used to display the system localized to
RTL languages (Hebrew/Arabic/Persian etc.).
Kind regards,
Yaron Shahrabani.
--
Ticket URL: <http://transifex.org/ticket/451#comment:9>
Transifex Trac
Reporter: lashar | Owner: alup
Type: defect | Status: assigned
Priority: major | Milestone: 1.0.1
Component: View/Templates | Version:
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
* cc: transifex@… (added)
--
Ticket URL: <http://transifex.org/ticket/451#comment:10>
Transifex Trac
Reporter: lashar | Owner: alup
Type: defect | Status: assigned
Keywords: rtl | Easy_task: 1
Blockedby: | Blocking:
Ready_for_review: 1 | Review:
Needs_patch: 0 |
-----------------------------------+----------------------------------------
* milestone: 1.0.1 => 1.2
--
Ticket URL: <http://transifex.org/ticket/451#comment:11>
Transifex Trac
Type: defect | Status: assigned
Priority: major | Milestone: 1.2
Component: View/Templates | Version:
Blocked By: | Blocking:
Easy Task: 1 | Needs Patch: 0
Ready for Review: 1 | Patch Review:
-----------------------------------+-------------------------
Comment (by jmphilippe):
Please also try to take into account the problem of automatic translation
in case of a mix between Latin characters and RTL characters. This is
typically happening when the text to be translated is containing codes
like %s or %d, or when the code is using tags to change rendering in the
application GUI.
We made some trials today with an Arabic speaker and ended with inverted
slash position in tags (eg. <em/> instead of </em>) when activating RTL in
the Html page code. We are not sure about which one is correct but feel
that we should still see </em> in the Arabic RTL text.
--
Ticket URL: <http://transifex.org/ticket/451#comment:12>
Transifex Trac
Reporter: lashar | Owner: alup
Type: defect | Status: closed
Component: View/Templates | Version:
Easy Task: 1 | Needs Patch: 0
Ready for Review: 1 | Patch Review:
-----------------------------------+-----------------------
* status: assigned => closed
* resolution: => wontfix
Comment:
Hi,
We are shutting trac down, so the issue has been moved to
https://github.com/transifex/transifex/issues/4.
Thanks,
Apostolis
--
Ticket URL: <http://transifex.org/ticket/451#comment:13>