Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

[BUG-225395] [FIXED] [EEP] "Use Region Settings" function has changed & name no longer makes sense #4206

Open
sl-service-account opened this issue Sep 2, 2018 · 1 comment

Comments

@sl-service-account
Copy link

Rider asked for this issue to be filed so he didn't forget.
This confused both Alexa & myself in the same way.

  • World -> Environment -> Use Region Settings.

  • On legacy viewers, "Use Region Settings" (World -> Sun -> Use Region Settings) always sets windlight to region/estate setting for sky/water/day cycle.

  • Even on Firestorm, which already has it's own version of parcel windlight, "Use Region Settings" always loads the region windlight & overrides any active parcel windlight.

  • On EEP, "Use Region Settings" will set to the parcel windlight if one has been set and will set to region windlight if a parcel windlight has not been set.

  • On EEP, parcel windlight always overrides region windlight.
    (Region < Parcel < Local < What you are editing)

  • Rider suggests renaming the "Use Region Settings" setting to "Use Region or Parcel Settings". or "Use Simulator Settings"

  • My personal preference would be to keep the "Use Region Windlight" setting having the legacy behaviour, so when a user chooses to apply "Use Region Windlight", they will always load the region windlight for each region they visit & parcel windlight settings will be ignored.
    Some people may not like windlight to constantly change when traversing lots of small parcels.
    Maybe rename the setting to "Always Use Region Windlight".

  • And add another setting under World -> Environment called "Use Region or Parcel Settings" that will load parcel windlight if available & region windlight if not.

Original Jira Fields
Field Value
Issue BUG-225395
Summary [FIXED] [EEP] "Use Region Settings" function has changed & name no longer makes sense
Type Bug
Priority Unset
Status Accepted
Resolution Accepted
Labels whirly-eep
Reporter Whirly Fizzle (whirly.fizzle)
Created at 2018-09-02T07:16:57Z
Updated at 2020-06-15T15:23:03Z
{
  'Build Id': 'unset',
  'Business Unit': ['Platform'],
  'ReOpened Count': 0.0,
  'Severity': 'Unset',
  'System': 'SL Viewer',
  'Target Viewer Version': 'viewer-development',
  'What just happened?': 'lalalal',
  'What were you doing when it happened?': '....',
  'What were you expecting to happen instead?': '....',
  'Where': 'secondlife://Aditi/secondlife/EEPTesting/249/55/23',
}
@sl-service-account
Copy link
Author

Whirly Fizzle commented at 2018-09-19T00:07:32Z

This is fixed on Second Life Test 5.1.9.519798 (64bit).
Setting is now called "Use Shared Environment"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant