Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

Display runtime version in deployment launcher #1302

Merged
merged 3 commits into from
Feb 25, 2020

Conversation

jamiebrynes7
Copy link
Contributor

@jamiebrynes7 jamiebrynes7 commented Feb 24, 2020

Description

Added a label which diplays the current runtime version & a button which opens the GDK Tools Configuration window to the deployment launcher.

image

Tests

  • Changed the runtime override and observed the change in the deployment launcher.

Documentation

@improbable-prow-robot improbable-prow-robot added jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/S Denotes a PR that changes 15-39 lines, ignoring generated files. labels Feb 24, 2020
@jamiebrynes7 jamiebrynes7 force-pushed the feature/display-runtime-depl-launcher branch from 57381f0 to 8c2f3cb Compare February 24, 2020 13:26
{
using (new EditorGUI.DisabledScope(manager.IsActive))
{
var buttonIcon = new GUIContent(EditorGUIUtility.IconContent(BuiltInEditIcon))
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it making this every frame?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm good question, to be honest - I cargo-culted this from the other buttons in this window. I can try making them readonly/static and seeing what happens

Jamie Brynes added 3 commits February 25, 2020 11:30
moved all style elements to a separate DeploymentLauncherWindowStyle
class

this class is lazily instantiated in OnGui()

this is the same pattern that is used for the
BuildConfigurationWindowEditorStyle

changed made based on the feedback from @zeroZshadow
@jamiebrynes7 jamiebrynes7 force-pushed the feature/display-runtime-depl-launcher branch from 0fd0704 to 1dee85c Compare February 25, 2020 11:51
@improbable-prow-robot improbable-prow-robot added size/L Denotes a PR that changes 150-299 lines, ignoring generated files. and removed size/S Denotes a PR that changes 15-39 lines, ignoring generated files. labels Feb 25, 2020
@jamiebrynes7 jamiebrynes7 merged commit 7a20be5 into develop Feb 25, 2020
@jamiebrynes7 jamiebrynes7 deleted the feature/display-runtime-depl-launcher branch February 25, 2020 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/L Denotes a PR that changes 150-299 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants