borevitzlab / timeplayer-9000

HTML5 timelapse player
MIT License
3 stars 0 forks source link

Make it more obvious which resolution the user is in (high/low) #4

Closed TimeScience closed 7 years ago

TimeScience commented 7 years ago

Icon changes but the button tool text doesn't update. Maybe an onscreen text next to the current time text on the right? can we just list the resolution (I'm not sure what is more useful)

gdunstone commented 7 years ago

changed the title-text.

gdunstone commented 7 years ago

@TimeScience is this enough? The tooltip now indicates whether it is hires or lores.

gdunstone commented 7 years ago

resolution added to infobox along with the number of images in the load queue. https://github.com/borevitzlab/timeplayer-9000/commit/71c8cbdea5ae1ff0d92aa4c0a49ddb051a38924b

TimeScience commented 7 years ago

that's better, thanks.

I like the idea of listing how many images need to be loaded, but it is confusing when the text is left at "0@lowres". Better text would be something like

Loaded: ###/### | Lowres/HiRes (or drop the "Loaded" text and just have the ###/### text)

-- ​ Tim Brown Research Fellow, Center for Plant Energy Biology Australian National University, Research School of Biology, Canberra ACT 0200 bit.ly/Tim_ANU http://bit.ly/Tim_ANU | tim.brown@anu.edu.au ​ +61 0419-006-102 ​ (m) | ​ Skype: TimeScience

On Fri, Feb 10, 2017 at 3:38 PM, Gareth Dunstone notifications@github.com wrote:

resolution added to infobox along with the number of images in the load queue. 71c8cbd https://github.com/borevitzlab/timeplayer-9000/commit/71c8cbdea5ae1ff0d92aa4c0a49ddb051a38924b

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/borevitzlab/timeplayer-9000/issues/4#issuecomment-278855810, or mute the thread https://github.com/notifications/unsubscribe-auth/AC7OxHOPdh7fFRUsZrbA2Us3ADqVocasks5ra-mvgaJpZM4L6LfA .

gdunstone commented 7 years ago

Im removing the entire counter because its confusing for everyone.

gdunstone commented 7 years ago

done.