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

[Installer] add back button into installer Welcome page #993

Closed
wants to merge 1 commit into from

Conversation

pythcoiner
Copy link
Collaborator

@pythcoiner pythcoiner commented Mar 4, 2024

fixes #306

i'm not really satisfied about the button position (any advice welcome), but at leat its functionnal.

image

@pythcoiner pythcoiner changed the title [Installer] add back button into installer first step [Installer] add back button into installer Welcome page Mar 4, 2024
@pythcoiner pythcoiner force-pushed the issue_306 branch 2 times, most recently from a5bd001 to bcaa28b Compare March 4, 2024 14:30
Copy link
Collaborator

@jp1ac4 jp1ac4 left a comment

Choose a reason for hiding this comment

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

I've tested this locally and it's working nicely, with the Back button only shown if I don't pass the network by command line.

I've added a couple of comments, but I think @edouardparis will be able to judge better the approach.

@darosior
Copy link
Member

darosior commented Mar 9, 2024

(any advice welcome)

Maybe the line below the three "Select" buttons, in the same style as the select buttons, centered?

@pythcoiner
Copy link
Collaborator Author

(any advice welcome)

Maybe the line below the three "Select" buttons, in the same style as the select buttons, centered?

you mean move here?

image

@darosior
Copy link
Member

darosior commented Mar 11, 2024 via email

@pythcoiner
Copy link
Collaborator Author

here the Previous button moved below slects button:

previous

@darosior darosior requested a review from edouardparis April 19, 2024 06:20
@jp1ac4 jp1ac4 mentioned this pull request May 29, 2024
@nondiremanuel
Copy link
Collaborator

This has been done in #1098
I'm going to close this PR

@pythcoiner pythcoiner deleted the issue_306 branch May 30, 2024 11:01
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.

GUI installer: add a previous button
4 participants