Skip to content

Gui editor fixes#5416

Merged
bramkragten merged 3 commits intodevfrom
gui-editor-fixes
Apr 2, 2020
Merged

Gui editor fixes#5416
bramkragten merged 3 commits intodevfrom
gui-editor-fixes

Conversation

@bramkragten
Copy link
Copy Markdown
Member

@bramkragten bramkragten commented Apr 2, 2020

Fixes #5413

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@bramkragten bramkragten added this to the 0.108.0 milestone Apr 2, 2020
@property() public lovelace?: LovelaceConfig;
@property() private _config?: StackCardConfig;
@property() private _GUImode = true;
@property() private _guiModeAvailable? = true;
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.

Does this need the optional (undefined) ? (I don't remember what that is called)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The event does not have to send it, so it could be undefined, should not happen...

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.

ah Gotcha


@property() private _saving: boolean = false;
@property() private _error?: string;
@property() private _guiModeAvailable? = true;
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.

Same here

@bramkragten bramkragten merged commit 4150ac0 into dev Apr 2, 2020
@bramkragten bramkragten deleted the gui-editor-fixes branch April 2, 2020 15:47
@bramkragten bramkragten mentioned this pull request Apr 3, 2020
@lock lock bot locked and limited conversation to collaborators Apr 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Code editors switch back to visual editor after just one keystroke

3 participants