search

metasfresh / metasfresh-webui-frontend-legacy

metasfresh Webui Frontend
GNU General Public License v2.0
57 stars 35 forks source link

Refactor/proper fix DocumentLists #1057

Open teosarca opened 7 years ago

teosarca commented 7 years ago

AIM of this task is to get rid of current hardcoded things around views, included views etc.

This task was triggered by https://github.com/metasfresh/metasfresh-webui-frontend/issues/1046 , and mainly by https://github.com/metasfresh/metasfresh-webui-frontend/pull/1055#pullrequestreview-50852765 were a workaround/hardcoded solution was provided.

Before starting this task, we need to elaborate a raw plan about what actually needs to be done.

wiadev commented 7 years ago

This partially improved with #1069

includedViews, quickActions and selections are currently correctly handled.

However it badly needs to be further improved in following aspects:

  1. Eliminate forced refreshes with refresh parameter containing timestamp/random value. This is extremely bad approach
  2. Optimize data flow to prevent re-rendering of component which don't have focus and not directly affected by last action. In particular optimize progress notifications flow
  3. Optimize reset of selection
  4. Optimize handling of sorting and filtering
  5. Reduce memory consumption by limiting affected scope after Redux actions
  6. Improve overall rendering performance
  7. Ease detecting of context within components (inModal, isModal, inBackgroud etc)