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

On-going Pinwheel SN mini updates #102

Open
29 of 35 tasks
patudom opened this issue Jun 7, 2023 · 5 comments
Open
29 of 35 tasks

On-going Pinwheel SN mini updates #102

patudom opened this issue Jun 7, 2023 · 5 comments
Labels
5. lower priority Would be nice but not critical

Comments

@patudom
Copy link
Contributor

patudom commented Jun 7, 2023

Suggested updates

Release:

  • Publicize new SN Mini on Twitter (John)
  • Post about new SN Mini on SciAct channels (Pat)

Constellations button:

  • When active, pause "play" functionality of time slider (else it keeps bringing you back to M101 FOV
  • Zoom out to a large enough FOV that you can see the constellations
  • Constellation icon should change color when focused like other buttons

M101 image handling:

Light curve:

  • Connecting line should correspond to displayed data points
  • y-axis & label should have 100% opacity and possibly have a thicker line weight (maybe match the weight of the timeline) and have a larger font
  • y-axis should be relabeled "Supernova Brightness"
  • Try making y-axis label horizontal for readability. (We can change it back to vertical if we hate it).
  • Add an x-axis arrow with a "Time" label
  • Hide light curve on a first pass through the images, then pop up option to display lightcurve.
  • Check with Mary & Harry - should we adjust position of div so y-axis actually connects to the timeline as its x-axis? (Pat)
  • Make button to show graph available from the beginning
  • When the time slider loops back to the beginning, it should start on 5/15. (Right now it loops to 5/16).

Controls:

  • Turn off functionality where the app tries to figure out your location and use it
  • Allow user to toggle whether to show pointer arrow or not.

Design:

  • Make sure selected colors meet accessibility contrast requirements (Pat)
  • Allow arrow to have rotation (Jon)
  • Give all buttons that can have "on/off" state the same behavior and propagate that to old Minis (Jon)

Content:

  • Update text (John)
  • Make a video (John)
  • Host video on YouTube and link from there (so it doesn't load so slowly and so we can track viewing numbers)
  • Discovery gallery image is taking a long time to load for Pat. (It's currently hosted at astrometry.net and that isn't being loaded immediately. We should host them in our own assets folder.)

Mobile issues:

  • Tapping on any of the buttons brings up tooltip and it won't go away until you turn function "off" (so "Watch the supernova change over time!" tooltip blocks the graph and image - change to "press and hold" to activate tool tips.
  • Constellation icon is sizing badly on mobile (and Safari)
  • M101 image needs to be placed higher up in interface, so the lighcurve doesn't cover the image.

Housekeeping:

  • Remove vestigial code (around location stuff) that we aren't actually using here. (Jon)
  • Bundle in Highway font so users don't need to have it installed (Jon)
  • Change folder/workspace name from m101-supernova to pinwheel-supernova (to de-jargonify the url)

Originally posted by @patudom in #98 (comment)

@johnarban
Copy link
Collaborator

M101 "play" mode addressed in #105

#104 connects the Y-axis with the time slider

@Carifio24
Copy link
Member

#107 handled the on/off button state, including for old minis

@Carifio24
Copy link
Member

#115 allows us to rotate the arrow annotation.

@Carifio24
Copy link
Member

#107 turned off the button tooltips on mobile, and #116 brings them back via a long press.

@johnarban
Copy link
Collaborator

#122 shows the button at start, updates the text, and fixes the playback functionality

#120 toggles the arrow, but it's behaves a little oddly for me

@patudom patudom changed the title On-going SN mini updates On-going Pinwheel SN mini updates Jul 7, 2023
@patudom patudom added the 5. lower priority Would be nice but not critical label Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5. lower priority Would be nice but not critical
Projects
None yet
Development

No branches or pull requests

3 participants