-
Notifications
You must be signed in to change notification settings - Fork 209
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
Various demo UI fixes - checklist #232
Comments
Here's where the Save button is made: image-sequencer/examples/lib/defaultHtmlStepUi.js Lines 94 to 96 in 630eb77
We can use the same kinds of selectors to modify it, add more text, change the styling -- using Bootstrap styles: |
Breaking these up to make some first timer issues!! |
awesome!!!
…On Mon, Jul 16, 2018 at 7:34 AM Varun Gupta ***@***.***> wrote:
Breaking these up to make some first timer issues!!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#232 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJzq6k_uDhOGTU7cP_8E_h6HbCDWLks5uHHodgaJpZM4TuEVK>
.
|
Hello, I made changes to show 'Press save to see changes' next to each step's 'Save' button and add a 'download' button next to the 'Add Step' button. Please take a look. Pull request #325. |
I think this issue is done, closing this up now! |
Small but helpful list of tweaks we could break out into first-timers-only issues! (once the demo refactor is done (#214)
Save
button until step inputs have been changedAdd step
button until a module is selectedPress save to see changes
next to each step'sSave
buttondownload
button at the bottom, in a panel next to theAdd Step
buttonThe text was updated successfully, but these errors were encountered: