Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework weather options #110

Merged
merged 10 commits into from
Dec 17, 2021
Merged

Rework weather options #110

merged 10 commits into from
Dec 17, 2021

Conversation

ghost91-
Copy link
Contributor

No description provided.

@ghost91- ghost91- force-pushed the weather-options-rework branch 4 times, most recently from d645512 to 8cc8fbb Compare December 15, 2021 02:33
@ghost91- ghost91- changed the title feat: rework weather options Rework weather options Dec 15, 2021
Additionally, on failed world migrations, effects are disabled to
prevent any potential scene crashes.
@ghost91- ghost91- merged commit 3e4a39d into master Dec 17, 2021
ghost91- added a commit that referenced this pull request Dec 17, 2021
  * There are now sliders for every weather option (except tint).
  * The value of option sliders now updates in real time when the slider is moved.
  * `speed` is now a relative factor for the base speed of the effect
  * The base speed and scale of effects are now adjusted according to the grid size
  * `density` now represents the maximum number of particles per grid unit.
  * Existing scenes (also in compendiums) are migrated automatically for the new interpretations of
    the weather options.
  * The `updateWeather` and `switchWeather` hook events have been deprecated. They are still
    available and expect options in t    he old format. Going forward, the `fxmaster.updateWeather`
    and `fxmaster.switchWeather` hook events should be used instead. Th    ey expect the new option
    format.
@ghost91- ghost91- deleted the weather-options-rework branch December 17, 2021 01:21
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.

None yet

1 participant