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

UI improvements to WebTSA #236

Closed
3 tasks done
SRGDamia1 opened this issue Apr 30, 2018 · 5 comments
Closed
3 tasks done

UI improvements to WebTSA #236

SRGDamia1 opened this issue Apr 30, 2018 · 5 comments
Assignees
Labels
high priority ready for testing Issue has been resolved and deployed on sandbox for testing tested Issue has been tested on sandbox and is ready for production

Comments

@SRGDamia1
Copy link
Member

SRGDamia1 commented Apr 30, 2018

People over at Stroud really don't like the some of the features "missing" from WebTSA:

@SRGDamia1 SRGDamia1 added this to the Uploading data milestone Apr 30, 2018
@Maurier Maurier self-assigned this Jun 12, 2018
@Maurier Maurier added the WebTSA label Jun 12, 2018
@horsburgh
Copy link
Member

After discussing this on the phone last week, @Maurier suggested adding zoom capabilities to the upper plot using the mouse scroll wheel. Panning through time would still be done on the lower pane as it is now.

The ability to trace the line and see individual points may be more difficult to implement because the code actually "thins" the data to make it plot faster (e.g., when you show a very long time series and you are zoomed out, TSA does not plot every point because you could not visually perceive every point and it would take unnecessarily long time to draw).

@Maurier - is it still implemented this way? Is there a point at which you've zoomed in far enough that all of the points are plotted and you could trace the data? The implication is that when hovering you would only see the points that get plotted and not all of the points and so it might be misleading for users.

@horsburgh horsburgh added ready for testing Issue has been resolved and deployed on sandbox for testing and removed in progress labels Jul 25, 2018
@horsburgh
Copy link
Member

horsburgh commented Jul 26, 2018

These look really awesome. We had demos from @Maurier yesterday, and I just tested on the sandbox today. The following have been added:

  • Vertical zooming/scaling by hovering mouse over a y-axis and using the mouse scroll wheel or track pad to zoom in or out.
  • Vertical dragging by clicking on the values of a y-axis and dragging up or down
  • Horizonal zooming/scaling by hovering over the plot area and using the mouse scroll wheel or track pad to zoom in or out
  • Horizontal scrolling continues to be controlled via the thumbnail plot at the bottom of the screen, but can also be controlled by clicking on the plot window and dragging left or right.
  • Individual data values are shown for all plotted time series by hovering over the plot area and dragging mouse left or right

Adding the "tested" label.

@horsburgh horsburgh added the tested Issue has been tested on sandbox and is ready for production label Jul 26, 2018
@aufdenkampe
Copy link
Member

These do look awesome.

There is one additional set of tweaks to the X-Axis (time), however, which I think is quite important.

The X-Axis labels make it really hard to see the time frame that you are looking at, especially the Year, which is not shown anywhere.

I recommend these changes to the X-Axis labels:

  • Add a global X-Axis label that includes Time Zone information, such as "DateTime in UTC" or preferably "DateTime in UTC-6 (Central Daylight Time)".
    • Presently timezone is not listed anywhere, and it seems that it is all in UTC. At a minimum, this needs to be listed somewhere. Optimally, we would display in local time or allow a user to set the time zone, but I understand that we might not be able to implement that now under current constraints.
  • Always show Year, but preferably not for every Tick mark. Can it be shown for just the first and last tick mark?
  • Remove days of the week from the Tick Labels. Not only are they not really necessary, but they make it much harder to see the Month labels, especially when zoomed in. Replace with Month.
  • Remove the hour Tick Labels entirely and just show the tick marks for hours. These labels are not really necessary given that the wonderful feature when hovering a mouse over the plot that shows all the values and the full timestamp for a given X value.

@Maurier, Let me know what you think, and if any of these changes can be easily made.

@horsburgh
Copy link
Member

@aufdenkampe - these are good ideas, but I'm going to ask @Maurier to hold off on any new development on TSA until we at USU decide what we can and can't do with other funding. I suggest moving this to a new issue as this one will get closed as soon as we've verified that the fix for the original issue as reported is deployed on production (which it should be now).

@horsburgh
Copy link
Member

Improvements to the TSA UI that meet the original issue are live on production. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority ready for testing Issue has been resolved and deployed on sandbox for testing tested Issue has been tested on sandbox and is ready for production
Projects
None yet
Development

No branches or pull requests

4 participants