17 replies to this topic

[Huevos]

https://github.com/O...omment-31625720

 

That commit was added a few days ago so we can skin the scroll bar. But that means we have to skin the scroll bar in every single screen. That is crazy and adds around 500 lines of code to the skin.

 

So, how can we skin the scroll bar once and use those settings globally throughout the whole skin?

Edited by Huevos, 11 December 2018 - 08:55.

[ims]

and "scrollbarSliderForegroundColor" parameter has no effect for SelectionList

[littlesat]

That commit was added a few days ago so we can skin the scroll bar. But that means we have to skin the scroll bar in every single screen. That is crazy and adds around 500 lines of code to the skin.

 

So, how can we skin the scroll bar once and use those settings globally throughout the whole skin?

For > 15 years nobody wants to modify that scrollbar... so there is no real big need to do so... ...

In addition it is not possible to make a template of a scrollbar and that is actually what needs to be done to do it with less code... 

The thinks in the parameter session are of course general. Of course it could be done that more defaults of the scrollbar are forwarded via the parameter 'session'.

Edited by littlesat, 11 December 2018 - 10:22.

[Dimitrij]

and "scrollbarSliderForegroundColor" parameter has no effect for SelectionList

Yes, users complained about this bug.

[betacentauri]

I'm quite sure that it worked in a Listbox.

[mrvica]

scrollbar width should be skinnable too, it is just too thick, it is still hard coded
https://github.com/O...elistbox.cpp#L9

[littlesat]

That’s the default... nobody yells for more than 15 years about this... so why is it such subject now?

Edited by littlesat, 11 December 2018 - 19:57.

[betacentauri]

Littlesat don't ask why. Every user and skinner is different. In most cases default scrollbar is ok for me, but in some cases it's quite nice to be able to change the scrollbar.

Question is whether there is somebody who is able to improve it and has time to do it.

 

I think I'm out. I have no idea what to do. The GUI and skin things are a closed book for me.

[mrvica]

in FHD skin is scrollwidth ok (20 pixels out of 1920), in HD skin it is quite thick (20 pixels out of 1280), we have nowadays large flat screens and no more CRT TV´s, no need for 20 pixels scrollbar

[Taapat]

@mrvica after these changes with the parameter scrollbarWidth you can change the scrollbar width: https://github.com/O...istbox.cpp#L627

[mrvica]

you mean to put in the skin like this

<parameters>
......................
<parameter name="scrollbarWidth" value="10" />
......................

well I need to recompile the image, do I?

[Taapat]

Huevos in the first post point to the commit.

But this is possible only in the  develop branch, whose images are not publicly available.
You need to add scrollbarWidth="10" to the scrollbar parameters:

For example:

<widget source="list" render="Listbox" position="10,10" size="355,300" scrollbarMode="showOnDemand" scrollbarWidth="10">

[littlesat]

That was indeed what I meant... with parameter arrange a new default setting... currently for each list it is a parameter....

Edited by littlesat, 11 December 2018 - 21:29.

[Huevos]

 

Huevos in the first post point to the commit.

But this is possible only in the  develop branch, whose images are not publicly available.
You need to add scrollbarWidth="10" to the scrollbar parameters:

For example:

<widget source="list" render="Listbox" position="10,10" size="355,300" scrollbarMode="showOnDemand" scrollbarWidth="10">

No that is changing the scrollbar on a screen by screen basis. That requires changes to hundreds of lines in the skin. To make skinning the scrollbar practical it should be possible to set the parameters in one place in the skin and those settings apply to every scrollbar that is not individually skinned.

Edited by Huevos, 11 December 2018 - 22:49.

[Taapat]

No that is changing the scrollbar on a screen by screen basis. That requires changes to hundreds of lines in the skin. To make skinning the scrollbar practical it should be possible to set the parameters in one place in the skin and those settings apply to every scrollbar that is not individually skinned.

 
I only answered mrvica how to set the scrollbar width using the current changes.
I understand your question, as already mentioned, we already had an internal discussion that we should implement a global setup for this, and maybe for all other local parameters.

Edited by Taapat, 12 December 2018 - 08:01.

[Huevos]

Well can we have the same discussion on the public forum so everyone can participate?

[littlesat]

Sure...

[ims]

 

I'm quite sure that it worked in a Listbox.

Should not be in it, that SelectionList(MenuList) using in final eListboxPythonMultiContent ?