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

Running and finished simulations are now reported to experimenters #192

Merged
merged 3 commits into from
Mar 13, 2015

Conversation

btruhand
Copy link
Collaborator

No description provided.

@btruhand
Copy link
Collaborator Author

Tell me if there's anything amiss Jorge. I know the UI's not great

@jorgearanda
Copy link
Owner

Thanks Btara, and sorry for the delay. I'll take a look at this pr tomorrow.

@jorgearanda
Copy link
Owner

Nice job, Btara. Besides the naming problems, the other thing I'd like you to polish a bit is the UI--to make it consistent with the style in the rest of the page. Thanks!

@btruhand
Copy link
Collaborator Author

Hi Jorge, sorry I missed the meeting. I was extremely fatigued from yesterday. About the UI, what do you think needs to be added/removed in particular so I have a grasp of what I need to do

@jorgearanda
Copy link
Owner

No worries, Btara. For the UI, try using the widgets that Bootstrap uses if you're not already, and perhaps try a table format like the rest of the tables in that page.

@btruhand
Copy link
Collaborator Author

So instead of making it a scrolling list you want me to make it into a table format? Would you want me then to let the table just continually grow, or do you want me to put a limit to the number of items shown there?

@jorgearanda
Copy link
Owner

Letting it continually grow is fine by now :-)

@btruhand
Copy link
Collaborator Author

btruhand commented Mar 3, 2015

Jorge, right now I have the information layed out side by side, one for running and finishing simulations the other planned to be for interrupted simulations. Do you want them to stay the same or have them be separated sections?

@jorgearanda
Copy link
Owner

I like the way it looks and does not clutter when it's side by side.

@btruhand
Copy link
Collaborator Author

btruhand commented Mar 7, 2015

I've made the changes Jorge and this PR also solves issue #144 . Also I've opened a new issue #201

jorgearanda added a commit that referenced this pull request Mar 13, 2015
Running and finished simulations are now reported to experimenters
@jorgearanda jorgearanda merged commit 51a123f into jorgearanda:master Mar 13, 2015
@jorgearanda
Copy link
Owner

Great, thanks!

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.

2 participants