Skip to content

Change order and wording snapshot actions#7677

Merged
bramkragten merged 7 commits intodevfrom
restore-wording
Nov 23, 2020
Merged

Change order and wording snapshot actions#7677
bramkragten merged 7 commits intodevfrom
restore-wording

Conversation

@ludeeus
Copy link
Copy Markdown
Member

@ludeeus ludeeus commented Nov 13, 2020

Breaking change

Proposed change

  • Makes the "Wipe & restore" button more clear,
  • Changes the order of the buttons

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 or discussion:
  • Link to documentation pull request:

Before:
image

After:
image

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:

@ludeeus ludeeus added the Supervisor Related to the supervisor panel label Nov 13, 2020
Co-authored-by: Bram Kragten <mail@bramkragten.nl>
}
.button-row {
display: flex;
align-content: space-between;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
align-content: space-between;
justify-content: space-between;

@bramkragten bramkragten merged commit 7f089c3 into dev Nov 23, 2020
@bramkragten bramkragten deleted the restore-wording branch November 23, 2020 16:08
@github-actions github-actions bot locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed Supervisor Related to the supervisor panel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants