source-academy / frontend / 5399648319 / 1

40%

master: 40%

push

github

Add agenda and stash visualisation to environment visualiser (#2532)

* Set up the base for starting with agenda stash visualization
* Checkbox to only display the agenda and stash
* Agenda and stash passed to EnvViz component
* Convert AgendaItems to their respective components with helper functions
* Very Basic Literal Value Item implemented

* Add generic Agenda Item Component

* Design for the stacks and generic stackItems implemented.
* Also did some refactoring and renaming of previous functions and components

* Add placeholder for each node and instruction

* Add display for binary/unary operators and for nodes using their program strings(not fully tested yet)

* Truncate text by width for agenda

* Use config ellipsis

* Fix bug where there was large space between item components

* Truncate text by height

* Integrate agenda and stash with environment
* only works for compact layout
* Shifted button group to the top so more area is available to display information

* Arrows from env instr to frmar

* Fixed bug with the arrows in the environment

* Fixed failing tetsts

* Modify setEditorHighlightedLines to highlight all lines from start to end

    * The saga for setEditorHighlightedLines was originally only
      highlighting the first line.
    * The reducer for setEditorHighlightedLines gave the correct state to
      the Editor component and added markers to the Ace Editor with
      className 'myMarker', but this class was not used.
    * The saga uses highlighting from inspector.js in public/externalLibs.
      We should remove that and use the reducer instead.

* Added line highlighting when hovering over agenda node

* The line highlighting probably needs a refactor. There isn't a way to
  unhighlight specific lines. It is being done by resetting the
  highlighted lines currently. We probably need to remember marker IDs,
  and have a way to highlight the gutter (which react-ace does not... (continued)

2011 of 6400 branches covered (31.42%)

Branch coverage included in aggregate %.

5335 of 13729 relevant lines covered (38.86%)

18.22 hits per line