2DegreesInvesting / tiltWebTool

https://bit.ly/tilt-app-info
https://2degreesinvesting.github.io/tiltWebTool/
GNU General Public License v3.0
1 stars 0 forks source link

Incorporate Tilman's suggestions into Webtool #213

Closed Tilmon closed 3 months ago

Tilmon commented 3 months ago

Dear @maurolepore,

responding to this comment , I reviewed the Webtool Document and the actual tool.

Based on the review, please do the following: Webtool Document

  1. In the webtool document, I made suggestions in track changes. Can you please incorporate the changes?

Webtool - general feedback sorted by tab

  1. Welcoming Page: Space between paragraphs seems larger for “Welcome to tilt’s Online Portal” card than for “What you’ll Find on This Website”. Is that true? If yes, can you please align the spacing?
  2. Dataset: When I logged in 20min ago, the data didn't load, I didn't even see the buffer "big data...". Now it worked again. Any idea what's the reason behind that?
  3. Dictionary: 2nd card was also empty when I logged in 20min ago, now it shows the actual dictionary. I guess this issue is somewhat related to the dataset not showing up (point 4). Just flagging here in case you have an idea what the reason might be.
  4. Method: This is just a heads up: We also need to upload a 2nd video once it's ready. Not sure yet when it will be ready, but ideally at some point next week. Just FYI.
maurolepore commented 3 months ago

WIP

RE 2.

Image

maurolepore commented 3 months ago

RE 2. I tested the app and confirm that after timeout the app looks gray, indicating the user that needs refresh. Refreshing results in logout. Loging back in brings the app back to life.

The timeout now is 300s (5'). We can change the settings easily.

maurolepore commented 3 months ago

Having explained 3-4, I take that as understood. I'll close but feel free to open a dedicated issue if we need some change.

Tilmon commented 3 months ago

Hi @maurolepore , sorry for late response!

RE 1, 2 and 5: Great!

RE 3 & 4: thanks for explaining and already extending the time out period. I just tested it again and for some reason, the data don't load when logging in. I also re-freshed, logged in again but still no data are automatically previewed (see screenshot). When I click on the "Go!" button, the data load. But they don't show up without hitting the "Go!" button. Is that how it's supposed to be? Dicitionairy works fine now, so extending the time out period seems to have worked there!

Image

maurolepore commented 3 months ago

@Tilmon

When I click on the "Go!" button, the data load. But they don't show up without hitting the "Go!" button. Is that how it's supposed to be?

Yes, that's how it's supposed to be. That avoids expensive computations before the user is done making all the selections they want. It gives the user more control and saves computation and costs.

But I do see your point. It would be nice to start with the app showing some data rather than an empty page.

This behavior has been like this for a while. If you still need to test the app it would be best to do it ASAP so we can prioritize those changes during sprint planning tomorrow.

Tilmon commented 3 months ago

Hi @maurolepore OK I think then it's good :) I think I got confused because I was used to seeing data immediately from the past. But if a user looks at this the first time, I'm sure they'll figure out to click "Go!" ;)