ricklancee / new-tab-myanimelist Goto Github PK

The current watching list is sorted by start dates first, but it would be an improvement to sort the list by what's airing first and then start dates. This will make sure two-cour shows are not falling behind shows that are finished.

Current, ACCA is finished but LWA is still airing.

The desired way would be LWA first since it's still airing.

When finished with watching a show the a card should prompt to complete the show. This should also set the completed at date (if it's not already set).

The app should have some form of localisation with localisation strings in a json file and toggleable with an option.

lang/
  en.json
  nl.json

Currently the way state is handled is done by passing through a global state (the list) and not always copied but passed as a reference within each component. This isn't the best way since changing the state object would result in the state changing within each component.

With no connection the list is still loaded from localstorage and images from disk (if cached), however some images might nog get loaded and an error is thrown that mal could not be updated.
The app should retry to fetch the list when connection is back up and show a different error when down.

• Retry connection when internet is back online
• Any changes made to the list should be saved and uploaded to mal when back online (check for changes on mal first)
• Possible use a service worker for cached images

If I give it my login credentials, I get this:

Remove moment js since it's HUGE and only used for displaying human readable dates diffs like
in 2 days which can be written manually.

Currently components have a lot of duplicate code, especially bootstrapping and passing dependencies. This should be done in a more modular way.

For instance create a factory that binds dependencies.

// component.js
function component({ bus, time }, selector, state) {
  ...
}

export default componentFactory(deps) {
  return component.bind(null, deps);
}

// shell.js
...
const component = componentFactor({ time, bus });
component('selector', state)

Components should also be composed together with some way of generic component bootstrapping to avoid having to write a register function for each component.