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

Update layout of the launch-wizard #918

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

shanedell
Copy link
Contributor

Update layout of the launch-wizard

  • Move the program and data fields to be the first two options.
  • Moved around some other items.
  • Renamed title of "Program" to "Main Schema File"
  • Renamed title of "Data" to "Input Data File"

Closes #846

NOTE: Later on once we get the root element issue, #653, fixed we will need to this setting to the launch wizard.

- Move the program and data fields to be the first two options.
- Moved around some other items.
- Renamed title of "Program" to "Main Schema File"
- Renamed title of "Data" to "Input Data File"

Closes apache#846
Copy link
Contributor

@mbeckerle mbeckerle left a comment

Choose a reason for hiding this comment

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

+1
I did not run it to see what it looks like, but nothing in these changes suggests any problems/issues with this change.

Copy link
Collaborator

@stricklandrbls stricklandrbls left a comment

Choose a reason for hiding this comment

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

+1

Reference Screenshot

image

@shanedell shanedell merged commit f31364c into apache:main Dec 18, 2023
21 checks passed
@shanedell shanedell deleted the launch-wizard-layout-change branch December 18, 2023 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Revisit layout of launch wizard
3 participants