Springen naar inhoud


Foto

InputDeviceSetup Lenght

inputdevicesetup

  • Please log in to reply
Er zijn 7 reacties in dit onderwerp

#1 athoik

  • PLi® Core member
  • 8458 berichten

+327
Excellent

Geplaatst op 19 oktober 2013 - 06:53

Hello,


InputDeviceSetup has a hardcodec lenght of 400. This causes issues with some translations (eg the Greek one), letters are not shown even though there is enouph space.

Changing length to 600 solves the above issue.

Bijgevoegde Bestanden


Wavefield T90: 0.8W - 1.9E - 4.8E - 13E - 16E - 19.2E - 23.5E - 26E - 33E - 39E - 42E - 45E on EMP Centauri DiseqC 16/1
Unamed: 13E Quattro - 9E Quattro on IKUSI MS-0916

Re: InputDeviceSetup Lenght #2 athoik

  • PLi® Core member
  • 8458 berichten

+327
Excellent

Geplaatst op 19 oktober 2013 - 15:58

Hello,

On PluginList we are also limiting the plugin description (without apparent reason) although there is space.

Bijlage  0001-PluginList-make-some-space-for-longer-plugin-descrip.patch.txt   997bytes   5 Aantal bijlagen

Before patch:

Bijlage  screenshotOld.jpg   24,87K   66 Aantal bijlagen

After path:

Bijlage  screenshotNew.jpg   25,33K   64 Aantal bijlagen

Thanks
Wavefield T90: 0.8W - 1.9E - 4.8E - 13E - 16E - 19.2E - 23.5E - 26E - 33E - 39E - 42E - 45E on EMP Centauri DiseqC 16/1
Unamed: 13E Quattro - 9E Quattro on IKUSI MS-0916

Re: InputDeviceSetup Lenght #3 mirakels

  • Forum Moderator
    PLi® Core member
  • 7599 berichten

+62
Good

Geplaatst op 19 oktober 2013 - 23:04

I guess this is not a problem in the with of the box. When a box has certain width and you start at xpos = 120 in the box, there is only width-xpos space left in the box. So if you xpos=120 and you use width-40 to draw text you basically write outside the box.

 

The the greek (and probably other languages) width issue may be in the font renderer...


Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: InputDeviceSetup Lenght #4 athoik

  • PLi® Core member
  • 8458 berichten

+327
Excellent

Geplaatst op 20 oktober 2013 - 10:10

But same happens with default skin PLi-HD and English (almost 42 chars dispayed).

Bijlage  screenshot.jpg   27,39K   28 Aantal bijlagen

So does that mean that English font renderer suffers from the same issue?

Veranderd door athoik, 20 oktober 2013 - 10:12

Wavefield T90: 0.8W - 1.9E - 4.8E - 13E - 16E - 19.2E - 23.5E - 26E - 33E - 39E - 42E - 45E on EMP Centauri DiseqC 16/1
Unamed: 13E Quattro - 9E Quattro on IKUSI MS-0916

Re: InputDeviceSetup Lenght #5 mirakels

  • Forum Moderator
    PLi® Core member
  • 7599 berichten

+62
Good

Geplaatst op 20 oktober 2013 - 10:39

Hm, in the download list it shows that all width is used.
So maybe something else is wrong.

In the code the default width is 440 pixels, but in the skin 630 pixels are reserved. I expect the 630 is passed to the widget, but maybe it does not.
Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: InputDeviceSetup Lenght #6 athoik

  • PLi® Core member
  • 8458 berichten

+327
Excellent

Geplaatst op 20 oktober 2013 - 11:29

I add the following to PluginList class.
        print "PluginEntryComponent width:" + str(width)
And this is what i get when i am opening the list:
PluginEntryComponent width:440
...
PluginEntryComponent width:440
PluginEntryComponent width:440
So probably we are not passing 630, but the default 440 used.
Wavefield T90: 0.8W - 1.9E - 4.8E - 13E - 16E - 19.2E - 23.5E - 26E - 33E - 39E - 42E - 45E on EMP Centauri DiseqC 16/1
Unamed: 13E Quattro - 9E Quattro on IKUSI MS-0916

Re: InputDeviceSetup Lenght #7 mirakels

  • Forum Moderator
    PLi® Core member
  • 7599 berichten

+62
Good

Geplaatst op 20 oktober 2013 - 11:38

yes already found the problem. Just commited the fix.

Other list may suffer from the same problem, who knows...


Geen wonder... Had slechts een dm7000, maar wel ook een rotor. eigenlijk al een tijdje ook een dm600 en dm7025. Maar nu kijkend met een et9000 en vuduo

Re: InputDeviceSetup Lenght #8 athoik

  • PLi® Core member
  • 8458 berichten

+327
Excellent

Geplaatst op 20 oktober 2013 - 11:40

I can find PluginEntryComponent only in PluginBrowser and we are calling it without width argument.
$ find -type f -name "*.py" | xargs fgrep PluginEntryComponent
./lib/python/Components/PluginList.py:def PluginEntryComponent(plugin, width=440):
./lib/python/Screens/PluginBrowser.py:          self.list = [PluginEntryComponent(plugin) for plugin in self.pluginlist]
So what you suggest to do? Change PluginBrowser or Change PluginEntryComponent?

Edit1. Just show your post. Thanks!

Veranderd door athoik, 20 oktober 2013 - 11:41

Wavefield T90: 0.8W - 1.9E - 4.8E - 13E - 16E - 19.2E - 23.5E - 26E - 33E - 39E - 42E - 45E on EMP Centauri DiseqC 16/1
Unamed: 13E Quattro - 9E Quattro on IKUSI MS-0916


1 gebruiker(s) lezen dit onderwerp

0 leden, 1 bezoekers, 0 anonieme gebruikers