Skip to content
This repository was archived by the owner on Jun 11, 2020. It is now read-only.

SetCore TopbarEnabled - #267

Merged
SolarCrane merged 1 commit into
masterfrom
Luke
Jan 25, 2016
Merged

SetCore TopbarEnabled#267
SolarCrane merged 1 commit into
masterfrom
Luke

Conversation

@Stickmasterluke

Copy link
Copy Markdown
Contributor

Allows developers to call StarterGui:SetCore("TopbarEnabled",false) to turn off the topbar and all core UI except the setting hamburger icon.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since other functions in Playerlist are using the '.' instead of ':' syntax and because Playerlist is a singleton we should continue the convention and use a '.' here for the function declaration. Please revise other scripts to match this change.

@bmanre2

bmanre2 commented Jan 22, 2016

Copy link
Copy Markdown

THIS IS THE HOLY SAVIOR OF ALL DEVS!

@Gskartwii

Copy link
Copy Markdown

+1

Comment thread CoreScriptsRoot/CoreScripts/Topbar.lua Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if disableable is a word, can we rename this variable so that it is a little more understandable?

FF for TopbarEnabled and Minor Adjustments

Cleanup for Defeatable Topbar
SolarCrane added a commit that referenced this pull request Jan 25, 2016
@SolarCrane
SolarCrane merged commit d4a8601 into master Jan 25, 2016
@bmanre2

bmanre2 commented Jan 26, 2016

Copy link
Copy Markdown

yASSS!!! IT HAS BEEN DONE!

@Zero-Bits

Copy link
Copy Markdown

Great, so when will it be in-game?

@Gskartwii

Copy link
Copy Markdown

@Zero-Bits As it was already merged, I would think right now, or at least soon enough.

@Zero-Bits

Copy link
Copy Markdown

Looks like it's not available yet in studio.

@Gskartwii

Copy link
Copy Markdown

I'm going to say that the "FFlag" is not set yet, but don't quote me on that.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants