This is a list of all comments for CR-2888.

Review Summary: No summary

General Comment by Matthias Claessens on 18 August 2025, 07:42 
http://subversion02:8060/cru/CR-2888#c4493

  Voeg je de JSON bestanden van het front en frontreling ook nog
  onder SVN?
  Deze ontbreken nog volgens deze commit, en ik zie ze ook niet
  in SVN.

  Reply by Joannes Lemmens on 18 August 2025, 12:02
  > Ik heb ze toegevoegd op svn en ook aan deze review toegevoegd
  > mocht je ze nog willen bekijken.

General Comment by Matthias Claessens on 18 August 2025, 13:00 
http://subversion02:8060/cru/CR-2888#c4496

  prima, ziet er goed uit.


----------------------------------------
File: development/Visualisatie_3D/Parametric_3D_models/MVX.json
Revision Comment by Matthias Claessens on 18 August 2025, 07:36 
http://subversion02:8060/cru/CR-2888#c4492

  Heel dit element is praktisch een duplicaat van het
  DesignElementFront Low, met als enige verschil de hoogte van
  het element.
  Zou het niet praktischer zijn als je deze combineert en de
  hoogte parametrisch maakt?
  
  {{"HeightGlassFrontPanel = is(FrontTypeHeightM, 'Glass') ? 52 :
  is(FrontTypeHeightC, 'GlassLow') ? 70 : 138",}}
  
  zoiets, maar dan kan je hiervoor je condities van
  {{ShowFrontDesingElementHigh}} nog voor gebruiken.

  Reply by Joannes Lemmens on 18 August 2025, 13:03
  > Goede opmerking, ik heb ze gecombineerd


---

ID: CR-2888 http://subversion02:8060/cru/CR-2888

Title: [ICT-5147] 3D-MVX: Front
Statement of Objectives:
[ICT-5147]

State: Closed

Summary: 

Author: Joannes Lemmens
Moderator: Joannes Lemmens
Reviewers: (0 active, 1 completed*)
     Matthias Claessens   (*)