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.

Sign up for GitHub

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

Jump to bottom

Fix selective UI script error when changing certain other settings #3872

Merged
merged 1 commit into from
Jun 5, 2016
Merged

Fix selective UI script error when changing certain other settings #3872

merged 1 commit into from
Jun 5, 2016

Conversation

jonpas
Copy link
Member

@jonpas jonpas commented Jun 4, 2016

When merged this pull request will:

  • Add check if a UI class even exists in SettingChanged event
  • Fix script error when changing certain other settings:
21:18:53 Error in expression <ex = ace_ui_elementsSet find [_element, !_show];
if (_index != -1) then {
if (_s>
21:18:53   Error position: <!_show];
if (_index != -1) then {
if (_s>
21:18:53   Error Generic error in expression
21:18:53 File z\ace\addons\ui\functions\fnc_setAdvancedElement.sqf, line 38

@jonpas jonpas self-assigned this Jun 4, 2016
@jonpas jonpas added this to the 3.6.0 milestone Jun 4, 2016
@jonpas jonpas added the kind/bug-fix Release Notes: **FIXED:** label Jun 4, 2016
@jonpas jonpas assigned jonpas and unassigned jonpas Jun 4, 2016
@thojkooi thojkooi merged commit 2ad3f57 into master Jun 5, 2016
@thojkooi thojkooi deleted the fixUIerror branch June 5, 2016 11:44
@jonpas jonpas mentioned this pull request Jun 5, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jonpas jonpas

Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Milestone
3.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@jonpas @thojkooi