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
rosie go: improve new suite wizard behaviour. #1338
rosie go: improve new suite wizard behaviour. #1338
Conversation
This gives a larger, maximizable window for editing the suite info data (access-list, title, etc). It also makes sure that the project metadata is correctly loaded for extra compulsory variable addition. The metadata for our standard rose-suite.info options has also been improved.
@@ -40,7 +40,7 @@ help=Specify a space-separated list of ticket/issue ids. | |||
= #32 #45 | |||
|
|||
[=opts] | |||
description=A space-delimited list of optional configuration keys to switch on. | |||
description=A space-delimited list of opt conf keys to switch on. |
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.
Think I preferred the original expanded version of this since it is effectively documentation.
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.
It's too long for a description - it pushes the buttons etc too far to the right.
Looks good to me. Works as expected with the demo metadata available. Just those two comments. |
@benfitzpatrick - bump. |
@benfitzpatrick - bump. |
Looks good to me. Manually testing out as working. |
…d-tweaks rosie go: improve new suite wizard behaviour.
This gives a larger, maximizable window for editing the
suite info data (access-list, title, etc). It also makes
sure that the project metadata is correctly loaded for
extra compulsory variable addition, and that changes
in the edit window do not persist when going back
and changing the project.
The metadata for our standard rose-suite.info options has
also been improved.
@arjclark, please review.