Skip to content

Instantly share code, notes, and snippets.

@mdarby

mdarby/things.md Secret

Last active May 10, 2018 14:11
Show Gist options
  • Save mdarby/f049d005dcdceba16dc1f9befe815670 to your computer and use it in GitHub Desktop.
Save mdarby/f049d005dcdceba16dc1f9befe815670 to your computer and use it in GitHub Desktop.

Misc

  1. Should the Terms of Use link open to a new tab?
  2. The DS Select modal height is % of page. When page is tall it looks weird. When page if short it hides the Select button/link.
  3. Clicking on the Workflow Optimization Tool link in the top nav returns a user to the Sub-Domain Root_path as if no DS has been selected (The page is empty) and the DS selector is not presented. We should either A) return them to the DS they already selected at login, or B) Present the DS select modal again. The only way to get it populated now is to refresh the page manually and then select a DS again. This should be fixed as of https://github.com/arcadia/workflow-app/pull/136
  4. Should the slideout menu close after making a selection from any of the dropdown selectors contained in it?
  5. Need some event to trigger the page to update after the No. of Enounters slide is used.
  6. We need to add a link to the User section of the slide out for the User Guide once it is created and hosted somewhere. Arcadia will have to provide that link to MP
  7. Slideout menu range slider will have to have the max value tied to the Number of Encounters max value. It is currently just hard coded at 300.
  8. Trend arrow colors need to be updated to use $trend-up-arrow color variable for both up and down values. The variable can be updated to just be $trend-arrows or something like that.
  9. Metrics card heights are a bit too tall. Need to reduce the top and bottom padding to better match the HiFi mockup design.
  10. Metrics selections - Need to highlight the selected metric card in the top by making it’s background color blue and have a checkmark in the top right of the card (this would match the HiFi mockups)
  11. Whichever metric is selected needs to be highlighted in the tooltip. Shown in the HiFi Mockup.

Heatmap

  1. Add AVG Column. This column is to display the average value of the selected metric. The rows are also to be sorted by this AVG column when the graph is built and displayed
  2. Number of days needs to be tied to the month (28, 30 or 31)
  3. As the metric is selected from the dropdown in this section the graph needs to update values, AVG column, highlight the metric card in the top of app, and highlight the select metric in the tooltip.
  4. The tooltip text values needs to match the actual metric. If metric is Encounters Per Day the text in the tooltip needs to read the same. Currently just says Encounters
  5. Need to add Vertical scroll to the graph. This would need to have the x-axis locked at the top. They do not want the tab to grow in height for large data sets. Here is something that might help with this one: http://bl.ocks.org/lmatteis/895a134f490626b0e62796e92a06b9c1 This should be fixed as of https://github.com/arcadia/workflow-app/pull/136
  6. Cells should narrow up as the page gets narrow. The container min-width is 900px so it will not get smaller than that. Right now the grid cell widths are calc’d based on the values of margin and width when the graph loads. Need to have it in CSS so it will just update as the screens are smaller.
@croz1007
Copy link

croz1007 commented May 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment