search

IM3S / LDAR_Sim

MIT License
2 stars 0 forks source link

Web App Content Review #155

Open c-vollrath opened 2 years ago

c-vollrath commented 2 years ago

Hi @soroushojagh - I reviewed the web app content and here are some thoughts and suggestions below. Overall it's looking amazing - nice work!

Home page

  1. Under "What is LDAR-Sim?" we say "multiple different technologies." I would delete "different" and say "LDAR programs may consist of multiple technologies."
  2. Need a period at the end of the last sentence under the "What is LDAR-Sim?" section.
  3. I think the link to the education page is super cool!
  4. Under "How does the web app work" the word credential should be plural. On the setup and run part I would say "programs are simulated on our servers." On the view results part I would say "View and download results from simulations to understand how LDAR programs performed."
  5. Under the "Key features" section for configurable programs you could write "Designing custom LDAR programs requires flexibility. Users can adopt default parameters or configure highly customized programs." Partnerships should be plural in the built together part. Estimate is spelled wrong in the designed for everyone box.

About page

  1. In the second paragraph, I think the acronym for IM3S should be expanded to demonstrate what it stands for the first time it's mentioned. A semi-colon can also be used after the acronym before you say "a collaboration...".
  2. There is a y on Alberta Energy Regulator that should be dropped.
  3. There is no link provided to GitHub at the end where it says "click here." But I'm sure this is intentional right now ;).

Education

  1. The clickable preview boxes look really good! Can we take screenshots of the publications that we're going to host on this page? That way users could see an Environmental Science & Technology paper or Environmental Research Letters paper as the preview. I think this would look cool!

How to use:

  1. Should there be a mix of text and videos? For example, the steps could be written out in order with supporting videos below each step. It could be presented as a flowchart or something.

Setup - programs

  1. The drop down menu looks great.
  2. When I clicked "modify" and hovered over number of simulations and process with my cursor, a message appeared that said "you cannot change this," but I think these parameters are changeable.
  3. When I modified a program and clicked save, the app would keep me in the general settings tab - it wouldn't return me to the initial page where all parameters are listed for that program unless I selected a different program from the drop down menu and then re-selected the program I just edited. I'm sure you're actively working on this though!
  4. In the general settings tab, "number of process" should be "number of processes."
  5. The program reports are sweet!!!

Other notes:

The app is looking awesome @soroushojagh! Let me know if you have any questions about the feedback - I hope it's helpful!

tybob-gough commented 2 years ago

I've looked it over as well and only have one minor edit to add to Coleman's list: On the setup, run, and results pages, when not logged in, a message says “without registration or logging in , we are unable to display setup menus. Please Sign Up here or Login here”. This might be simpler as: “LDAR-Sim cannot be used without an account. Please Sign Up here or Login here.”. There is currently a link for logging in but no link for signing up.

This looks amazing - really great work.

tbarchyn commented 2 years ago

Login and sign up - sign up button is missing if browser window screen is narrow and tall. Terms are missing. Couldn't login - network error. Apologies I could not provide more feedback - I can go at this again when I can register.