Disabling UI

[Regret]

Why does disabling UI using F5 disable LuaUI drawscreen?

[imbaczek]

do i infer correctly this breaks Lua effects?

[Regret]

yes (actually not sure if all)

[Auswaschbar]

Which ones?
Commander nametags -> work
Display DPS -> works
LUPS -> works

[Regret]

Which ones?
Commander nametags -> work
Display DPS -> works
LUPS -> works

drawscreen

All those use drawworld etc afaik.

[imbaczek]

isn't drawworld the place you should be drawing effects, anyway?

edit: i've got a feeling that you actually want a different callin with a working name of DrawScreenUIHidden.

[Regret]

isn't drawworld the place you should be drawing effects, anyway?

edit: i've got a feeling that you actually want a different callin with a working name of DrawScreenUIHidden.

No, there already is a check to see whether UI is disabled, I don't see the point to force drawscreen disabled.

Spring.IsGUIHidden

[Auswaschbar]

isn't drawworld the place you should be drawing effects, anyway?

edit: i've got a feeling that you actually want a different callin with a working name of DrawScreenUIHidden.

No, there already is a check to see whether UI is disabled, I don't see the point to force drawscreen disabled.

WHy would someone call DrawScreen if screen drawing is disabled anyway?

Btw, you also fail to give any example widget...

[Regret]

WHy would someone call DrawScreen if screen drawing is disabled anyway?

Btw, you also fail to give any example widget...

To change cursor size.

http://springrts.com/phpbb/viewtopic.php?f=23&t=19934

[imbaczek]

No, there already is a check to see whether UI is disabled, I don't see the point to force drawscreen disabled.

1) clean separation of responsibility
2) backwards compatibility

[Regret]

1) clean separation of responsibility
2) backwards compatibility

Drawscreen was always enabled. The disabling when UI is hidden was introduced only in recent spring versions. So what backwards compatibility are you talking about?

Also widget makers are responsible for checking whether UI is hidden with Spring.IsGUIHidden().

[imbaczek]

no idea, i didn't design the api nor do i follow changes in it; it's how i'd do it. if it was working differently than it is now, there should be a rationale somewhere.

[Regret]

no idea, i didn't design the api nor do i follow changes in it; it's how i'd do it. if it was working differently than it is now, there should be a rationale somewhere.

What on earth is the Spring.IsGUIHidden for then? It was always used in drawscreen. There is no rational explanation for forcing drawscreen to be disabled other than catering to lazy widget makers that do not make the said check.

[Auswaschbar]

Spring.SetMouseCursor should be called from widget:Update anyway (see trepans SmoothScroll-widget).

[Regret]

Spring.SetMouseCursor should be called from widget:Update anyway (see trepans SmoothScroll-widget).

It does not work in Update.

EDIT: seems there is widget:DrawScreenEffects() which makes this topic nil.