Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace SWALs with Interactive Dialogs #1161

Conversation

ArturoManzoli
Copy link
Contributor

  • Added 'auto close on timeout' function to InteractionDialog
  • Small changes on dialogs UI

Closes #1117

@rafaellehmkuhl
Copy link
Member

rafaellehmkuhl commented Jul 30, 2024

This PR also fixes #291.

Copy link
Member

@rafaellehmkuhl rafaellehmkuhl left a comment

Choose a reason for hiding this comment

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

We are missing a width definition in the BlueOS conflict dialog:

image

Comment on lines 115 to 118
}
export enum MAVLinkManualControlButton {
R0 = 'BTN0_FUNCTION',
Copy link
Member

Choose a reason for hiding this comment

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

Bad indentation here.

constructor(public id: MAVLinkButtonFunction, public name: string) {}
constructor(public id: MAVLinkButtonFunction, public name: string) {}
Copy link
Member

Choose a reason for hiding this comment

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

Bad indentation also here.

@ArturoManzoli ArturoManzoli force-pushed the 1117-Cockpit-should-use-only-interactive-dialogs branch 3 times, most recently from d7db265 to 5572809 Compare July 30, 2024 17:20
}
}

export enum MAVLinkManualControlButton {
Copy link
Member

Choose a reason for hiding this comment

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

This indentation was right. There's no need to touch it.

Copy link
Contributor Author

@ArturoManzoli ArturoManzoli Jul 30, 2024

Choose a reason for hiding this comment

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

That's weird..
My linter must be crazy

Ill check that

Copy link
Member

@rafaellehmkuhl rafaellehmkuhl left a comment

Choose a reason for hiding this comment

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

There are still some indentation problems.

@ArturoManzoli ArturoManzoli force-pushed the 1117-Cockpit-should-use-only-interactive-dialogs branch 6 times, most recently from 521f16b to 8510e88 Compare July 31, 2024 16:58
@ArturoManzoli ArturoManzoli force-pushed the 1117-Cockpit-should-use-only-interactive-dialogs branch from 8510e88 to fcf0760 Compare July 31, 2024 17:59
@rafaellehmkuhl rafaellehmkuhl merged commit 6d5cb4b into bluerobotics:master Jul 31, 2024
8 checks passed
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.

Cockpit should only use alerts from InteractionDialog component, since it uses our visual language
2 participants