Skip to content

Add more menu items - #117

Merged
Black4Blade merged 1 commit into
somini:masterfrom
Bluscream:patch-1
Aug 17, 2019
Merged

Add more menu items#117
Black4Blade merged 1 commit into
somini:masterfrom
Bluscream:patch-1

Conversation

@Bluscream

Copy link
Copy Markdown
Contributor

Like docs and gameserver/apikey management

Like docs and gameserver/apikey management
@halbrd
halbrd requested a review from Black4Blade August 17, 2019 00:49
@halbrd

halbrd commented Aug 17, 2019

Copy link
Copy Markdown
Collaborator

Looks good to me 👍 Thanks for your work!

@Black4Blade

Copy link
Copy Markdown
Collaborator

Looks really good to me, Thanks for the work, looks good

@Black4Blade
Black4Blade merged commit 65902c7 into somini:master Aug 17, 2019

@Black4Blade Black4Blade left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great work, looks good, adds great function to the skin

Black4Blade added a commit to Black4Blade/Pixelvision2 that referenced this pull request Aug 17, 2019
@Bluscream

Copy link
Copy Markdown
Contributor Author

woah, hold on. It wasn't supposed to be merged, at least not without a proper discussion. The new menu items should be merged into one so it doesn't take so much space for "normal" users.

@Bluscream
Bluscream deleted the patch-1 branch August 17, 2019 20:22
@Black4Blade

Copy link
Copy Markdown
Collaborator

I cant see it really harm much
But guess you right on that, can change it maybe to only show when someone set up on developer mode when the console shows up

Don't think it will harm much whatever the case is

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.

3 participants