Skip to content

Miscellaneous fixes#1983

Merged
diogotr7 merged 9 commits intoantonpup:devfrom
Wibble199:fixes/miscellaneous
Apr 28, 2020
Merged

Miscellaneous fixes#1983
diogotr7 merged 9 commits intoantonpup:devfrom
Wibble199:fixes/miscellaneous

Conversation

@Wibble199
Copy link
Collaborator

A few small fixes for some of the newer things:

  • Prevented delay evaluatable allowing negative delays.
  • Added an upper limit on the radial layer's segment count (90, which should be more than enough).
  • Marked most properties on the particle layer as overridable.
  • Fixed the particle layer's region not working properly after a preset was applied.
  • Fixed an the global region editor being not removing regions from the view when they have been externally changed (e.g. by the particle layer's presets).

Wibble199 and others added 8 commits April 25, 2020 10:31
Set at 90, which means a maximum arc of 4 degrees. Even at around 20 segements though, it's still barely noticable.
This is more to prevent people crashing it by setting a studpidly high number.
…ence was changed in the keysequence control by a binding.

This commit fixes the issues with a region not being removed from the layer editor when a preset is applied on the particle layer and also the issue with the checkbox not being set correctly initially on the radial layer control.
For future reference for anyone reading this: When needing custom code to run when a dependency property's value is updated, this should always be done in a dedicated changed handler, NOT the property setter method. This is because changes to bindings do not actually set the property and thus would not invoke custom code on the property setter.
Now that there is no dropdown saying what type an evaluatable is, the labels need to make it clear instead.
@diogotr7 diogotr7 merged commit 11d2eb9 into antonpup:dev Apr 28, 2020
@Wibble199 Wibble199 deleted the fixes/miscellaneous branch April 28, 2020 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants