Create steady-state, simple dynamic, and advanced dynamic input views (via JT)

taborlab / Iris

The Light Program Interface

BSD 3-Clause "New" or "Revised" License

12 stars 2 forks source link

Create steady-state, simple dynamic, and advanced dynamic input views (via JT) #263

Closed SynbioLucas closed 8 years ago

SynbioLucas commented 8 years ago

Breakdown:

@BrianLandry : steady-state view input HandsOnTable, angular data structure updates
@SynbioLucas : state transitions between the views, ensure display updates properly & variables are set
@felixekn : style new View buttons (bb4bd41acdc4929d42551551ebf11b6b84417d1f)

SynbioLucas commented 8 years ago

Transitions are all finished as of b0562003594272fbb2d425e1c9986455e288423b.

Just need to set the inputs and display properties for the Simple Dynamic view and should be good to go.

SynbioLucas commented 8 years ago

@felixekn I created a custom class (selected) that you can use to style the input style transition buttons more easily, to indicate which state Iris is currently in. (6b7abc5082ce72d6141076f766c09e8155b3a505)

@BrianLandry I also added a simple confirmation dialog box that forces the user to confirm that they're ok with all their data being erased, which only happens when you press 'OK'. It's just the default browser dialog, but it'll work for now. We can find a prettier solution using Bootstrap or something later if we really care. (6b7abc5082ce72d6141076f766c09e8155b3a505)

BrianLandry commented 8 years ago

@SynbioLucas In simple dynamic mode, what do you think about also hiding this portion of the display as well:

It doesn't provide any additional information and can confuse the user.

SynbioLucas commented 8 years ago

I didn't want to hide those at the time, but agree with you now. The only reason to keep that stuff would be to allow the user to minimize it, and I don't think that's important in Simple view. I'll fix this and make a commit on the inputStyles branch.

@BrianLandry Also, as of 311dfc4 SS view looks good, but I did notice that deleting a 0 (entered by default) will cause an error in the plate. What do you think about making blank cells automatically set to 0?

SynbioLucas commented 8 years ago

Closed as of 2b0fd88da1fcbfdb50d1c4320f4f9026a58d3dd7 .