Display issue on dynamic transition page

Issue #105 resolved
Stefan Ritt created an issue

If one starts a fresh experiment from an empty ODB, the transition page shows a "tr_status" at a place were it should not be.

Screen Shot 2017-11-22 at 16.07.30 .png

People also asked me what

Last transition: transition, Status: status (errorstr), Time:time

means. Maybe the best would be to hide these strings until they make sense (after the first transition).

When I start a run (without any other program running), the text area says

Last transition: Starting run..., Status: 1 (Success), Time: 0

where the string "Starting run..." might indicate that the start is still in progress (because of the three dots). The right thing to display there would be

Last transition: Start run, Status: Success, Time: 0

where one could argue if the "Time" display is of general user interest or only for expert debugging purposes, in which case it should not be shown.

Comments (7)

  1. dd1

    Yes, Thomas also requested that I change the text to read "transition data not available at the moment, it will become available after you yadda yadda yadda". K.O.

  2. Stefan Ritt reporter

    Why don't you just keep it blank. I guess everybody can understand that. Better than having to read lots of yadda yadda text.

  3. dd1

    transition data is now saved to ODB /system/transition. so the web page will only look strange if they start with an empty odb and until they start the first run. after that, odb has good data and the web page will show it, there will be no strange content. closing this bug. K.O.

  4. Stefan Ritt reporter

    You cannot mark a thread resolved if there are two requests which are not satisfied. Actually an issue should be resolved by the person who opened it, not the person who worked on it.

    So I went and put the tr_status filed into the "Run Status" table which makes the display a bit more homogenous. Then I changed the text to what Thomas suggested. Then I removed the "updateStatus" filed which was not really used. I replaced the only left occasion with dlgAlert to show an RPC error. I removed also the 95% table width, so the page now is closer to our status page. Not really sure what is better (have empty space left and right of a table, or stretch the table all the way to the border and have empty space inside the table), so if you dislike it switch it back. I adjusted a bit the padding inside the cells for better readability.

    Now I consider this thread being closed.

    Screenshot 2019-02-06 at 14.02.56 .png

  5. Log in to comment