Video-hub-app: Last steps before v2.0.0 release

Created on 11 Jan 2019  路  5Comments  路  Source: whyboris/Video-Hub-App

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_

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

_Dark Mode_

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

_Code_

  • [x] 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)
  • [x] Remember each view's zoom level #16
  • [x] Consolidate the view-switching bug (some views currently don't show)
  • [x] Check that all views handle window resizes and search-bar hiding
  • [x] Make sure that adding a rating, scrolling away, and scrolling back persists the rating in the view
  • [x] Remove most console.log statements
  • [x] Add new keyboard shortcuts (for new views) & update the list in settings
  • [x] Remove jankiness from the resolution, time, and rating filters

_Misc_

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

鈿狅笍 Note -- unfinished tasks moved to #234

2.0.0

Most helpful comment

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! 馃殌 鈿★笍

All 5 comments

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! 馃槩

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 馃憣

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! 馃殌 鈿★笍

馃槶 馃榿

Unfinished tasks moved to #234

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cal2195 picture cal2195  路  6Comments

whyboris picture whyboris  路  3Comments

whyboris picture whyboris  路  3Comments

cal2195 picture cal2195  路  4Comments

cal2195 picture cal2195  路  5Comments