|
|
Please help the devs to help you by giving useful info. What do you mean "latest SVN"? Please specify an except revision number. How do you know it is "latest SVN"? Have you tested the revision before it? If not, please specify which revision you used that did work, thanks. |
|
|
It worked in Revision 43539 but it is broken in Lazarus Revision: 43585 |
|
|
Can you disect this range so you get the actual revision that broke it? |
|
|
I don't know how to do that. I know that a patch was applied recently. Maybe it has something to do with issue 0025408 |
|
|
For my understanding: You say the patch "broke" BiDi for TCheckListbox. I think BiDi was not working before 0025408, the patch only did not fix text writing direction. Is that correct? |
|
|
> I don't know how to do that It's easy, but takes some time. Take the last revision that worked (you say: 43539) Take the first revision you noticed was broken (you say: 43585) Now take the middle of those two: 43562 Do make clean svn up -r 43562 Rebuild Lazarus, see if problem has gone, if so the problem is between 43539 and 43526, if not the problem is between 43526 and 43585 You continue this until you find the revison that broke it. It will take no more then 10 svn up's to disect through 1024 (2^10) revisions. You need only 46 revisions, so you should be fine with at most 6 times this process. |
|
|
@Bart: this is not needed. Avishai probably means r43576. But this revision doesn't change anything about text direction. He needs to answer wp's question. |
|
|
TextFlags_DT_RTLREADING.patch (616 bytes) |
|
|
@Avishai, please answer wp's question. @All: please test attached patch, maybe it solves this issue. |
|
|
Maybe I was wrong about it having worked in the past. It din't mirror before so it couldn't be uses in R2L apps and I didn't look at it often. Anyway the text direction is incorrect for R2L. |
|
|
I tested the patch with the new translation_demo (lazarus/examples/translation) to which I quickly added a CheckListbox. But using the modification or not does not make a difference, even when Hebrew language is selected. Interestingly, text direction in the Checklistbox is the same as in the checkbox on the same form. So, this would mean that RTL text direction is wrong for TCheckbox as well. Avishai, can you canfirm this? Could you run this demo and check? |
|
|
TCheckBox and TCheckGroup look correct but TCheckListBox has LeftToRight Text I added a screen shot show the problem. |
|
|
@Avishai: you forgot to attach it. |
|
|
CheckBoxes.PNG (4,727 bytes) |
|
|
Ha! I did everything except hit the upload button. Sorry about that. |
|
|
I just figured out where I got the idea that TCheckListBox BiDiMode Text worked in the past. It was from playing with Delphi. It works there. |
|
|
checklistbox-bidi.png (28,465 bytes) |
|
|
Applying Maxim's patch to Win32WSCheckLst.pas I am getting the attached result (checklistbox-bidi.png) which seems to be ok if I take Avishai's screenshot as a reference. (That BiDi thing is still mysterious to me...) |
|
|
That looks to be perfect wp. Basically, all it means is that the punctuation is placed at the left instead of the right when you use Latin chars. Everything else is the same. Of course with a R2L language it means that the chars go R2L. But the BiDirection is because the numbers (1,2,3) go LeftToRight just like in Western languages. My name would be .iahsivA |
|
|
I applied the patch. Please test and close if OK. If there will be no problems it will be merged to 1.2 in a couple of days. |
|
|
Awesome guys! Thank you for this. :) Especially thanks to wp. |
- Anonymous
