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

feat(ui): add custom js editor in create test screen (#465) #503

Merged
merged 2 commits into from
Oct 26, 2020
Merged

feat(ui): add custom js editor in create test screen (#465) #503

merged 2 commits into from
Oct 26, 2020

Conversation

alexandravassova
Copy link

new tab in step/scenario screen that opens JS editor, currently without backend functionality
Closes #465

@NivLipetz
Copy link
Member

NivLipetz commented Oct 18, 2020

@alexandravassova thanks alot for the PR! it's greatly appreciated! The icon inside the create test form looks great 😄
The only thing I'm a little skeptic about is when clicking on the custom JS button a new window opens up and it seems a little out of place. I have the following suggestion tell me what you think:

  1. I realize this wasn't mentioned in the issue but I think it would be best if we can have the custom JS inside the create test form underneath the body and above the expectations.
    Essentially here:
    image

Of course if this button isn't clicked this box would be not visible, and when clicked it will appear. It would be perfect if the button is clicked again when the box is visible it will disappear (but save the contents written)

  1. In the scenario tab I suggest to place it here:
    image

  2. Since we will have it inside the actual test form, we don't need the cancel and submit buttons that the original new window would open up with.

  3. Last small thing, the scenario tabs don't have a | divider in between the icons (js editor and duplicate) whereas in the step tabs there is a | divider, can you please add the divider also to the scenarios tab to make it even with the rest of the form?
    image

Again thanks a lot! I'm open for any questions/comments you have

@alexandravassova
Copy link
Author

ok, i am going to implement your suggestions, they make sense :)

@NivLipetz
Copy link
Member

@alexandravassova it looks great and exactly how I pictured it 😄 really nice job!
@manorlh will add his review this hopefully tonight or tomorrow at latest. Thanks a lot for the contribution!

@enudler
Copy link
Collaborator

enudler commented Oct 26, 2020

hi, @alexandravassova great work!
I'm going merge it to a feature branch as we need to implement the backend
I think it will count for hacktoberfest but it doesn't just let me know.

We will open an issue today for backend implementation, if you would like to participate LMK
Thanks again for the great PR!

@enudler enudler changed the base branch from master to feat/inline-js October 26, 2020 05:46
@enudler enudler merged commit 7138635 into Zooz:feat/inline-js Oct 26, 2020
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.

UI: Add custom JS editor in create test screen
4 participants