kbss-cvut / react-maintenance-planner

React component to visualize and manage maintenance plans.
GNU Lesser General Public License v3.0
0 stars 1 forks source link

Modify tasks that are not work orders #55

Closed LaChope closed 2 years ago

LaChope commented 2 years ago

Blue tasks on 2nd level are not work orders. Therefore, color should be different and specified in legend.

image

LaChope commented 2 years ago

@kalamartin

We need to define colors and names for different types of boxes in the diagram, each box is in different resource group. This is current assignment of resource groups

  1. Whole aircraft -- Phases (blue)
  2. Area -- What should be the name (propose color)?
  3. Group of people (resources) -- Schedules WO (red) - Maintenance WO (blue) - Task cards (green) ?
  4. Concrete person/mechanic -- Work session (yellow)

Please, can you check terminology, suggest colors / better names if needed so. If not clear, please comment.

kalamartin commented 2 years ago

1) Phases can stay blue. Eg Phase 1, phase 2.... 2) Boxes which have been considered as workorders but actually are "Taskcard type groups" can be with turquoise. 3) Areas are just areas, not needed to define more I guess. There are on the very left side and can be marked as you wanted by colour. 4) Resources ...ussually in excel sheet workplan, each scope has its own colour, so as the tasks in the workplan. Since each task (WO M, WO S and especially taskcards) belongs to specifics scope, it could be done as like that. Please advis if possible. Therefore color for TC, WO M and S wouldn´t be needed.

I guess rest is ok.

LaChope commented 2 years ago

Here are the updated colors along with the legend. @kalamartin can you check if legend is correct and if colors are ok? I left tasks cards in green and made WO M, WO S some gradient of green.

If you have any suggestion / comment let me know @kalamartin @blcham

image image

kalamartin commented 2 years ago

I would suggest Maintenance WO´s to be RED. The yellow color is easily to swap with Scheduled WO. Otherwise ok.

@LaChope

LaChope commented 2 years ago

Ok, thanks for feedback. Here is the new version:

image

@kalamartin

blcham commented 2 years ago

Looks nice.