-
Notifications
You must be signed in to change notification settings - Fork 254
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
Move scene param types to their respective fiat plugin scene files #5287
Conversation
Visual changes require screenshots in the PR. |
@@ -123,7 +123,7 @@ export const InfoDisplayScene = React.memo((props: Props) => { | |||
)) | |||
|
|||
return ( | |||
<SceneWrapper scroll hasNotifications> | |||
<SceneWrapper hasTabs scroll hasNotifications> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we're at it, fix the InfoDisplayScene
using deprecated components like MainButton
to use SceneButtons.
This should be a primary button (which SceneButtons
and all button group components enforce automatically)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to have screenshots in visual PRs so reviewers can catch these mistakes easier.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, thanks for tidying up
CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
noneRequirements
If you have made any visual changes to the GUI. Make sure you have: