Support and General Use > Theming and Appearance Customization

conditional viewports problems thread

<< < (3/3)

BdN3504:
I'm not asking for the commit to be backward compatible... If people who have created wps in the past and use the %pb syntax as i did, and try to adjust their wps to meet the new syntax, they will definitely stumble upon that same error. In the Customwps page it says nothing about the fact, that 0 as values for height and width are illegal. So if people get that error, they know what they have to change, is all i want to get across.

PaulJam:
In my opinion setting the width or height of the progressbar to zero is pointless, because it wouldn't be visible. I don't really think this needs extra mentioning.
But i think the WPS should just be rejected by the parser and not cause a crash.

Llorean:
Agreed, DivX0 should never show up. But I consider it a bug of the old tag that it accepted 0-width or 0-height bars in the first place.

robin0800:
The Progress Bar commit Uses |-| to indicate zero or the default

JdGordon:
I've just done another commit to the cabbie themes, so let me know if you notice anything bad happening with them... otherwise I tihnk we can close this thread...

Navigation

[0] Message Index

[*] Previous page