Restrict Graph to Viewport Bug

[Sarronnn]

[Sarronnn]

=> Same error as #945 ; looks like the root cause is the specific dependency versions that are installed when running a version of Node that is too old. But I want to double check with Dondi

[dondi]

Needs a reproduction check

[dondi]

Reproduction confirmed during meeting; @nchun2 will isolate the bug further and will post a comment detailing how it can be reproduced:

• Meeting reproduction was done on production; need to check if this also happens in beta
• Try other variations:
  • Loading demo from side panel
  • Loading network from database via menu/side-panel
  • Loading network from file via menu/side-panel
• When isolation comment is complete, @nchun2 can reassign to @ntran18

[nchun2]

The bug also occurs in beta when the "restrict graph to viewport" function is selected from the top menu, but not from the side menu. Occurs in both production and beta when the network is loaded from a database on either the top menu or the side menu. It seems that the bug occurs whenever the view function is selected from the top menu.

[dondi]

Great! I’m adding @ntran18 to this bug but will keep @nchun2 for now in case some clarification is needed for reproducing the bug

[ntran18]

The main problem with this is originally Restrict Graph to Viewport in the top menu has a class called "viewportOption", and whenever this class is change, it will update grnState.viewportSize. However, Restrict Graph to Viewport is not a viewport size, so there is no constant for that, and when passed to grnState.viewportSize, it is a null value, and the graph keeps changing the size.

[kdahlquist]

Confirmed that this is fixed in beta 7.0.2 on Firefox