-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update User Interface #46
Comments
Firstly apologies, i was waiting for your response but hadn't gotten the notification. Turns out I wasn't watching my own repository! I do like the look and overall feel of the redesign. I do agree that the layout could be improved to help enter data. I have a couple of comments or questions about the layout you are proposing:
Thanks again for your work. |
Hey, thank you for your response. Concerning your comments:
Two options here: Either through tabs inside the container or adding a row below the first eg. Pension.
Good point. I haven't thought about the header yet, but the save & load buttons could stay there, same for the links of course. For the results, I'd go for tabs.
Never used bootstrap before, but it's an easy solution. Not sure if Bootstrap is really what's impacting load times, might be some other JS going on. Either way is possible. My suggestion is using bootstrap as a start, see how it works with the option to scrap it later on in the process. Concerning the results: Let me know what you think and if you have any further thoughts, suggestions and feedback. |
In terms of bootstrap or not, I am not bothered either way. I just want to
ensure that we don't make things worse.
I am sure that the javascript libraries are not optimal and I hope to make
more progress in simplifying and cleaning up the code that will make this
better.
I think it is worth creating a branch and having a go at making changes.
Any time you think it is worth updating the main site, let me know and we
can see about merging the changes in.
If you have any questions, please let me know.
Thanks
Alistair Marshall
|
The User Interface looks a bit outdated. Updating the UI would increase the user experience and would make this project more distinguishable from the original cFIREsim.
I made a rough draft on how the redesign could look like, see attached. We'd have two columns left for data input and right for the simulation result. The right columns could stay fixed on desktop, so it's always visible. Would jump to the bottom for mobile.
Looking forward to hear your thoughts!
The text was updated successfully, but these errors were encountered: