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

Last steps before v2.0.0 release #102

Closed
21 tasks done
whyboris opened this issue Jan 11, 2019 · 5 comments
Closed
21 tasks done

Last steps before v2.0.0 release #102

whyboris opened this issue Jan 11, 2019 · 5 comments
Assignees
Labels
2.0.0 Priority for v2.0.0 release
Milestone

Comments

@whyboris
Copy link
Owner

whyboris commented Jan 11, 2019

This issue is to keep a list of tasks that should be done after all the new features we'd like to see in version 2.0.0 are added, but before the release occurs. Most of these tasks are on top of the issues tagged for Milestone v2.0.0

This is a good place to see just how much needs to be done to make the current master release-ready. In no particular order:

Visual

  • Reorganize the settings menu & tabs to be more coherent
  • Icon overhaul Icon overhaul #51
  • Confirm settings defaults - make the first app launch only display relevant buttons
  • Clean up the Miscellaneous view settings - group and describe buttons better
  • CSS consolidation: pull colors and some re-used elements from a central file
  • reevaluate colors - maybe better contrast for a11y (accessibility) and harmonize all the filter and tag colors
  • CSS consolidation: some components use the same CSS, so, reuse those to keep DRY
  • confirm animations look good; add animations on newly added features

Dark Mode

  • Move the dark mode button to just-before-last button in the toolbar
  • Confirm app responds to OS X Dark Mode settings / changes
  • Check the dark mode is working on all components; possibly simplify the CSS

Code

  • Streamline the view switching code - how element widths / heights are recomputed & how ng-scroll receives the expected heights / number of elements (we now have 6 views, used to be 3)
  • Remember each view's zoom level Remember Each View's Zoom Level #16
  • Consolidate the view-switching bug (some views currently don't show)
  • Check that all views handle window resizes and search-bar hiding
  • Make sure that adding a rating, scrolling away, and scrolling back persists the rating in the view
  • Remove most console.log statements
  • Add new keyboard shortcuts (for new views) & update the list in settings
  • Remove jankiness from the resolution, time, and rating filters

Misc

  • Update to latest Angular (v8)
  • Stress test the app with more than 1,000 videos in a hub

⚠️ Note -- unfinished tasks moved to #234

@whyboris whyboris mentioned this issue Jan 11, 2019
@whyboris whyboris self-assigned this Jan 19, 2019
@whyboris whyboris pinned this issue Jan 19, 2019
@cal2195
Copy link
Collaborator

cal2195 commented Jan 21, 2019

I think the full view scrolling issue has something to do with:
https://github.com/rintoj/ngx-virtual-scroller#items-with-variable-size

I tried enabling unequal child sizes, but the detail view started acting badly! 😢

@whyboris
Copy link
Owner Author

The only view that has variable sizes is the full-view

Somewhere in the code for the filmstrip, thumbnail, and file views I provide to the virtual-scroller the expected number of items to display (computed based on width and height of gallery view). This hasn't been done for the other views and so they are possibly wonky.

I'll review that part of code and find a solution for every view: it's easy enough to estimate how many elements will be displayed based on the app state 👌

@whyboris whyboris added this to the v2.0.0 milestone Feb 24, 2019
@whyboris whyboris added the 2.0.0 Priority for v2.0.0 release label Feb 24, 2019
@cal2195
Copy link
Collaborator

cal2195 commented May 15, 2019

Stress test the app with more than 1,000 videos in a hub

Phhffff 1,000? That's rookie numbers!
I can confirm it works flawlessly with 20,000+ videos in a single hub, over a network, with each video having ~20 manual tags added as well! 🚀 ⚡️

@whyboris
Copy link
Owner Author

whyboris commented May 15, 2019

😭 😁

@whyboris whyboris moved this from To do backlog to To do next in Boris's Current Workflow Jul 27, 2019
@whyboris whyboris mentioned this issue Jul 27, 2019
@whyboris whyboris moved this from To do next to In progress in Boris's Current Workflow Jul 28, 2019
@whyboris
Copy link
Owner Author

Unfinished tasks moved to #234

@whyboris whyboris moved this from In progress to In review in Boris's Current Workflow Jul 28, 2019
@whyboris whyboris unpinned this issue Jul 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0.0 Priority for v2.0.0 release
Projects
Development

No branches or pull requests

2 participants