Unconsistent behavior in textboxscrollup and defines in MP 1.13

Discussion in 'General Support' started by XMaM, January 5, 2016.

  1. XMaM

    XMaM Portal Member

    Joined:
    December 15, 2012
    Messages:
    39
    Likes Received:
    21
    Gender:
    Male
    Location:
    Asturias
    Ratings:
    +46 / 1
    Home Country:
    Spain Spain
    Show System Specs
    I think there is inconsistent behavior with defines in textboxscrollup (maybe introduced or not completely solved by MP1-4695). I will try to explain it (not easy for me).
    You can use <define> and <define property=...> in window XML file and in references.xml file, in different controls and different tags (<label>, <color>, <width>,...). Some combinations seem to work and others do not.
    Take for example this scenario:
    In references.xml file:
    Code (Text):
    1. <define>#test.define_text:This is an example define text</define>
    2. <define>#test.define_color:ffffffff</define>
    3. <define property="true">#test.property_text:This is an example property text</define>
    4. <define property="true">#test.property_color:ffffffff</define>
    5.  
    6. <style Name="style_test_control_define">
    7.    <label>#test.define_text_xml</label>
    8.    <color>#test.define_color_xml</color>
    9. </style>
    10. <style Name="style_test_control_property">
    11.    <label>#test.property_text_xml</label>
    12.    <color>#test.property_color_xml</color>
    13. </style>

    In any window XML file:
    Code (Text):
    1. <define>#test.define_text_xml:This is an example define text</define>
    2. <define>#test.define_color_xml:ffffffff</define>
    3. <define property="true">#test.property_text_xml:This is an example property text</define>
    4. <define property="true">#test.property_color_xml:ffffffff</define>

    Now, in the same window XML file, if you try to use define, property or style in textboxscrollup, some of them don't work.

    #test.property_text_xml is ok, as expected, but #test.define_text_xml don't work.
    #test.define_text and #test.property_text (both included in references.xml), neither works.



    If you use the style style_test_control_define in the textboxscrollup, doesn't work (the label or color text are not converted)
    If you use the style style_test_control_property in the textboxscrollup, it works, but also, if you try to override the tags (e.g. with <label>This is another text</label> in textboxscrollup), doesn't work.

    Trying to summarize:
    Code (Text):
    1.  
    2. define in references.xml      | don't work | can be overridden
    3. define in window xml          | don't work | can't be overridden
    4. property in references.xml | don't work | can be overridden
    5. property in window xml      | work           | can't be overridden
    6.  
    Can or can't be overridden refers to the use in defining a style in references.xml and then try to change in the window file.

    Wow, this is a mess :confused:. I hope this may be understandable, but I have tried to explain it as best I could (n).
     
    • Like Like x 1
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. XMaM

    XMaM Portal Member

    Joined:
    December 15, 2012
    Messages:
    39
    Likes Received:
    21
    Gender:
    Male
    Location:
    Asturias
    Ratings:
    +46 / 1
    Home Country:
    Spain Spain
    Show System Specs
    Someone can confirm that global defines and properties (<define> and <define property="true"..> included in references.xml file) are working in textboxscrollup and MP 1.13?
    I revert to 1.12, because I have not gotten it to work on my system and my skin was broken, so I can't do more tests.
     
  4. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,526
    Likes Received:
    4,734
    Ratings:
    +8,200 / 17
    Home Country:
    New Zealand New Zealand
    @Sebastiii
    Did this patch come from you, or...?
     
    • Thank You! Thank You! x 1
  5. Sebastiii
    • Team MediaPortal

    Sebastiii Development Group

    Joined:
    November 12, 2007
    Messages:
    16,236
    Likes Received:
    6,201
    Gender:
    Male
    Location:
    France
    Ratings:
    +10,198 / 9
    Home Country:
    France France
    Show System Specs
    Yes i think it was me (from i remember) it was a tricky to debug at first and i was helped from @wizard123 to trigger the issue, maybe it can help to reproduce it and see why, because the skin engine is really not my cup of tea (does the expression is the same as french lol ?)
     
    Last edited: January 13, 2016
    • Thank You! Thank You! x 1
  6. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,526
    Likes Received:
    4,734
    Ratings:
    +8,200 / 17
    Home Country:
    New Zealand New Zealand
    Yes! :D
     
    • Thank You! Thank You! x 1
  7. wizard123
    • Team MediaPortal

    wizard123 Design Group

    Joined:
    January 24, 2012
    Messages:
    2,499
    Likes Received:
    1,695
    Gender:
    Male
    Occupation:
    genius
    Ratings:
    +2,919 / 11
    Home Country:
    United Kingdom United Kingdom
    Using sebs patch/fix i was able to have a Latestmediadefines.xml in which i had defines like this

    <define property="true" evaluateNow="true">#pluginlist:playDisc,Trakt,OnlineVideos,Pictures</define>

    Without the patches, defines used in textboxscroll will never work and have not in previous versions either as the skin engine for textboxscrollup control does not parse them correctly as seb found.

    Not sure if the fixes were included in last release ?
     
    • Like Like x 1
    • Agree Agree x 1
  8. Sebastiii
    • Team MediaPortal

    Sebastiii Development Group

    Joined:
    November 12, 2007
    Messages:
    16,236
    Likes Received:
    6,201
    Gender:
    Male
    Location:
    France
    Ratings:
    +10,198 / 9
    Home Country:
    France France
    Show System Specs
    The MP1-4695 is included fro 1.13 Final :) but it seems to trigger some regressions then or maybe need another fix.
     
  9. wizard123
    • Team MediaPortal

    wizard123 Design Group

    Joined:
    January 24, 2012
    Messages:
    2,499
    Likes Received:
    1,695
    Gender:
    Male
    Occupation:
    genius
    Ratings:
    +2,919 / 11
    Home Country:
    United Kingdom United Kingdom
    Hmm not sure, i confirmed it was working but is the OP saying it does not work in MP1.13 or MP1.12 as he states he dropped back to 1.12 and its not working which it won't.

     
    • Like Like x 1
  10. XMaM

    XMaM Portal Member

    Joined:
    December 15, 2012
    Messages:
    39
    Likes Received:
    21
    Gender:
    Male
    Location:
    Asturias
    Ratings:
    +46 / 1
    Home Country:
    Spain Spain
    Show System Specs
    First of all, thank you all. I think I have not explained well (my english is not as good as I would like, if not bad ;)).


    Without the patch, <define property="true" evaluateNow="true">...<...> does not work in 1.12 in the <label> tag. With the patch, is working in 1.13 (at least in the <label> tag), so good work :). I do not remember now if it also worked in other tags in 1.13 if you insert "the define" in the window xml file (as <textcolor>#mycolor</textcolor>, but this was working in 1.12)

    However, if you put <define>#mycolor:ff000000</define> in references.xml and use it in skin xml file with <textcolor>#mycolor</textcolor>, does not work in 1.13. With MP 1.12, it worked perfectly. Another thing is when you use styles, but this is best left for another time, because perhaps it is related to the above.

    I think that the problem with defines in MP 1.12 was only in the <label> tag, while working properly in the other tags. Now, with 1.13, defines are working perfectly in the <label> tag, but it has stopped working in other tags.

    Maybe (I think that yes) this issue in aMPed skin is related with this.

    And again, thank you. I appreciate your work.
     
    • Like Like x 1
    • Informative Informative x 1
  11. wizard123
    • Team MediaPortal

    wizard123 Design Group

    Joined:
    January 24, 2012
    Messages:
    2,499
    Likes Received:
    1,695
    Gender:
    Male
    Occupation:
    genius
    Ratings:
    +2,919 / 11
    Home Country:
    United Kingdom United Kingdom
    Confirmed define in <textcolor> does not work but doubtful it's related to the patch as other controls like standard buttons are affected. Can not confirm it worked in 1.12 though.
     
    • Thank You! Thank You! x 2
    • Like Like x 1

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!