New MP1 skin test (2 Viewers)

[daWooky]

hi wiz, the epg border highlight option is acting like the opposite

option enabled-> <-


option disabled-> <-

 

[Sebastiii]

I just tried to replicate but all ok my end

Ah yes maybe that ajs change are included, I will try to look when back to home thanks

 

[Sebastiii]

Ok so no Ajs patch (no MP1-4831), i will try again against master

 

[Sebastiii]

Ok it seems that i have found why and maybe related to how is setup Windows 10 (need to confirm)
The error happen in below code when text value is why a "." but not with "," (without quote)

So for example, i have the text = (2.5,0) -> it will failed
But
If it's (2,5,0) -> no error

private void GetPosition(string text, ref float x, ref float y)
    {
      //      Log.Info("GetPos:{0}", text);
      x = y = 0;
      if (text == null)
      {
        return;
      }
      text = text.ToLowerInvariant();
      text = text.Replace("screencenterx", GUIGraphicsContext.OutputScreenCenter.X.ToString());
      text = text.Replace("screencentery", GUIGraphicsContext.OutputScreenCenter.Y.ToString());

      int pos = text.IndexOf(",");
      if (pos >= 0)
      {
        x = float.Parse(text.Substring(0, pos));
        y = float.Parse(text.Substring(pos + 1));
      }
      else
      {
        x = float.Parse(text);
      }
    }

 

[Sebastiii]

Ok so in that XML : (BasicHomePage3.xml)

The line 195 : we have 0.5 and it will create the issue here

<animation effect="zoom" start="100,100" end="0.5,100" time="150" center="85,1017">Hidden</animation>

I have that setting on my windows :

Changing it too :

And it works lol

 

[daWooky]

i have . as digit grouping symbol

 

[Sebastiii]

Good it seems that i have found a fix
We need to set MediaPortal.exe as cultureinfo US

Thread.CurrentThread.CurrentCulture = new CultureInfo("en-US");
Thread.CurrentThread.CurrentUICulture = new CultureInfo("en-US");

With this, the MP exe instance is set to use Us culture info so it will handle correctly the decimal
I have tested that the keyboard stay in french under MP and it works

So i think it's the good way to go because maybe other part of code need to be used like culture info US.

 

[daWooky]

US refugees

 

[Sebastiii]

@wizard123

I notice something (the progress seems odd lol) i didn't see the current position

 

[wizard123]

@wizard123

I notice something (the progress seems odd lol) i didn't see the current position

View attachment 184381 View attachment 184382

Which osd is that for? Do you have the latest files from git and clear cache because im sure i fixed that the other day