[Evaluate] - Fix RTL text scrolling

Discussion in 'Submit: code patches (MediaPortal/TV-Server/etc.)' started by bob987, April 7, 2010.

  1. bob987
    • Premium Supporter

    bob987 MP Donator

    Joined:
    January 25, 2009
    Messages:
    47
    Likes Received:
    5
    Ratings:
    +5 / 0
    Home Country:
    Israel Israel
    This patch fixes some of the problems concerning the scrolling of text with right to left characters in it.

    Currently RTL text scrolls in a very strange way (see Strange Hebrew text scroll )
    With this patch the RTL text will scroll continually to the right in a smooth way without jumping.



    This bug fix will improve all the RTL language users UI Experience



    There is still some work to be done concerning the RTL scrolling:
    A) In mixed text (LTR & RTL) when a new LTR word appears its characters will appear in the wrong order ("hello" will not appear h->e->l->l->o but o->l->l->e->h). This also the case with the current text scrolls.

    B) When the RTL scrolling text appears, the right most part of the text is already trimmed from the screen. The text should appear with its right most part aligned to the right side of the text box.

    C) RTL scrolling text should have the fade effect in the left side of the text box.
    I will try to solve A in the future. B+C are probably over my ability.

    Hope this will get into RC2,
    Bob
     

    Attached Files:

    • Like Like x 1
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. bob987
    • Premium Supporter

    bob987 MP Donator

    Joined:
    January 25, 2009
    Messages:
    47
    Likes Received:
    5
    Ratings:
    +5 / 0
    Home Country:
    Israel Israel
    Probably most of the dev guys don’t have a system with RTL language installed, so I made two short avi to let you see the bug and the fix.
    There are 3 scrolling texts:
    a) Only LTR.
    b) RTL + LTR
    c) Only RTL
    Movie recorded using horizontal scroll speed=4

    Bob
     

    Attached Files:

  4. bob987
    • Premium Supporter

    bob987 MP Donator

    Joined:
    January 25, 2009
    Messages:
    47
    Likes Received:
    5
    Ratings:
    +5 / 0
    Home Country:
    Israel Israel
    I've updated the patch to RC2 (no changes in the code) for my own use so I send the updated file.

    Is there some thing I can do to help this get into the next release?

    Bob
     

    Attached Files:

  5. muellers

    muellers Portal Member

    Joined:
    July 25, 2010
    Messages:
    12
    Likes Received:
    0
    Ratings:
    +0 / 0
    AW: Fix RTL text scrolling

    Hi,

    will this patch include in Release 1.2?

    greets
     
  6. elliottmc
    • Premium Supporter

    elliottmc Retired Team Member

    Joined:
    August 7, 2005
    Messages:
    14,927
    Likes Received:
    4,011
    Gender:
    Male
    Location:
    Cardiff, UK
    Ratings:
    +5,503 / 13
    Home Country:
    United Kingdom United Kingdom
    Show System Specs
    Hi,

    I can't test this myself, but from the videos provided, it seems that it would be good to include it.

    Perhaps one of the developers could build binaries for testing?

    As long as it doesn't cause any problems, it is probably safe to commit.

    Mark
     
  7. thecode

    thecode Portal Member

    Joined:
    March 4, 2011
    Messages:
    9
    Likes Received:
    3
    Ratings:
    +4 / 0
    Home Country:
    Israel Israel
    I've tried to test it with the SVN version r27641 but for some reason live TV stopped working for me (some index out of bounds)
    The patch does solve the scrolling bug, Since I've done some changes to my version also and had to merge files might be problem on my side.
    I'll try over the weekend to do clean patch on the latest SVN and publish my conclusions.

    thecode
     
    • Like Like x 1
  8. thecode

    thecode Portal Member

    Joined:
    March 4, 2011
    Messages:
    9
    Likes Received:
    3
    Ratings:
    +4 / 0
    Home Country:
    Israel Israel
    I tried testing the patch, it does work, but I'm sad to say it doesn't fix all situations.
    If the test has mixed RTL text and LTR text it will not scroll correctly at all.
    In mixed text it should scroll as it has only RTL text.
    I hope to have time to fix that, but if it is not fixed I suggest waiting with the patch (Altought is still fix some situations and doesn't cause problems).

    thecode.
     
  9. bob987
    • Premium Supporter

    bob987 MP Donator

    Joined:
    January 25, 2009
    Messages:
    47
    Likes Received:
    5
    Ratings:
    +5 / 0
    Home Country:
    Israel Israel
    Hi
    I know that this code only fixes RTL only texts.
    I tried to fix the mixed text situation but to do this there is a need to rewrite all the text handling code (which is over my capabilities).
    I do think that this patch will improve the current code and should be implemented in V1.2

    For V2, it will be a big improvement to use the internal window API for text printing. there is a very complex logic with mixed LTR + RTL text, and windows does a great job in this area.

    Bob
     
Loading...

Users Viewing Thread (Users: 0, Guests: 0)

  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.
    Dismiss Notice
  • About The Project

    The vision of the MediaPortal project is to create a free open source media centre application, which supports all advanced media centre functions, and is accessible to all Windows users.

    In reaching this goal we are working every day to make sure our software is one of the best.

             

  • Support MediaPortal!

    The team works very hard to make sure the community is running the best HTPC-software. We give away MediaPortal for free but hosting and software is not for us.

    Care to support our work with a few bucks? We'd really appreciate it!