Skip to content

Commit d58bb63

Browse files
committed
resolved merges from updated DEV branch
2 parents 1341940 + ba7c05c commit d58bb63

31 files changed

+267
-316
lines changed

package.json

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@
3333
},
3434
"contributors": [
3535
"Abaas Khorrami",
36+
"Alex Gomez",
37+
"Alexander Landeros",
3638
"Ali Rahman",
3739
"Andy Tsou",
3840
"Andy Wong",
@@ -43,52 +45,71 @@
4345
"Caner Demir",
4446
"Carlos Perez",
4547
"Chris Flannery",
48+
"Chris Guizzetti",
4649
"Christopher LeBrett",
4750
"Cole Styron",
51+
"Daljit Gill",
52+
"Dane Corpion",
53+
"David Bernstein",
4854
"David Chai",
4955
"David Kim",
5056
"Dennis Lopez",
57+
"Edar Liu",
5158
"Edwin Menendez",
59+
"Eivind Del Fierro",
5260
"Ergi Shehu",
5361
"Eric Yun",
5462
"Freya Wu",
5563
"Gabriela Jardim Aquino",
5664
"Gregory Panciera",
5765
"Haejin Jo",
66+
"Harry Fox",
5867
"Hien Nguyen",
5968
"Jack Crish",
6069
"Jackie Yuan",
6170
"James McCollough",
6271
"James Nghiem",
6372
"Jasmine Noor",
73+
"Jason Victor",
6474
"Joseph Park",
75+
"Joseph Stern",
6576
"Josh Kim",
6677
"Joshua Howard",
6778
"Kevin Fey",
6879
"Kevin HoEun Lee",
6980
"Kevin Ngo",
7081
"Kim Mai Nguyen",
82+
"Kris Sorensen",
83+
"Kristina Wallen",
84+
"Kyle Bell",
7185
"Lance Ziegler",
7286
"Lina Shin",
7387
"Mark Teets",
7488
"Mike Bednarz",
7589
"Minzo Kim",
90+
"Morah Geist",
7691
"Nathanael Wa Mwenze",
92+
"Nathan Richardson",
7793
"Ngoc Zwolinski",
7894
"Nick Huemmer",
7995
"Peter Lam",
8096
"Prasanna Malla",
97+
"Quan Le",
8198
"Rajeeb Banstola",
8299
"Raymond Kwan",
83100
"Robby Tipton",
101+
"Robert Maeda",
84102
"Rocky Lin",
85103
"Ruth Anam",
86104
"Ryan Dang",
87105
"Sergei Liubchenko",
106+
"Sean Kelly",
88107
"Sierra Swaby",
89108
"Tania Lind",
90109
"Viet Nguyen",
110+
"Vincent Nguyen",
91111
"Wilton Lee",
112+
"Yididia Ketema",
92113
"Yujin Kang",
93114
"Zachary Freeman"
94115
],

src/app/components/Action.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ const Action = (props: ActionProps): JSX.Element => {
3333
last, // boolean on (whether the view index is less than 0) AND if (the index is the same as the last snapshot's index value in hierarchyArr) in 'ActionContainer'
3434
index, // from snapshot.index in "ActionContainer's" 'hierarchyArr'
3535
sliderIndex, // from tabs[currentTab] object in 'ActionContainer'
36-
dispatch, // dispatch function from our storeContext in 'ActionContainer'
36+
dispatch,
3737
displayName, // from snapshot.displayName in "ActionContainer's" 'hierarchyArr'
3838
componentData, // from snapshot.componentData in "ActionContainer's" 'hierarchyArr'
3939
viewIndex, // from tabs[currentTab] object in 'ActionContainer'

src/app/components/Loader.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22

33
import React from 'react';
44
import { ClipLoader } from 'react-spinners';
5-
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
6-
import { faCheck, faExclamationCircle } from '@fortawesome/free-solid-svg-icons';
5+
import CheckCircleOutlineIcon from '@mui/icons-material/CheckCircleOutline';
6+
import ErrorOutlineIcon from '@mui/icons-material/ErrorOutline';
77

88
/*
99
This file is what decides what icon (loading, checkmark, exclamation point) is displayed next to the checks in the ErrorContainer loading screen:
@@ -15,9 +15,9 @@ This file is what decides what icon (loading, checkmark, exclamation point) is d
1515

1616
const handleResult = (result: boolean): JSX.Element =>
1717
result ? (
18-
<FontAwesomeIcon icon={faCheck} className='check' size='lg' /> // if result boolean is true, we display a checkmark icon
18+
<CheckCircleOutlineIcon className='check'/> // if result boolean is true, we display a checkmark icon
1919
) : (
20-
<FontAwesomeIcon icon={faExclamationCircle} className='fail' size='lg' /> // if the result boolean is false, we display a fail icon
20+
<ErrorOutlineIcon className='fail'/> // if the result boolean is false, we display a fail icon
2121
);
2222

2323
// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types

src/app/components/StateRoute/ComponentMap/ComponentMap.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ export default function ComponentMap({
3939
const [layout, setLayout] = useState('cartesian'); // We create a local state "layout" and set it to a string 'cartesian'
4040
const [orientation, setOrientation] = useState('vertical'); // We create a local state "orientation" and set it to a string 'vertical'.
4141
const [linkType, setLinkType] = useState('diagonal'); // We create a local state "linkType" and set it to a string 'diagonal'.
42-
const [stepPercent, setStepPercent] = useState(10); // We create a local state "stepPercent" and set it to a number '10'.
42+
const [stepPercent, setStepPercent] = useState(0.5); // We create a local state "stepPercent" and set it to a number '0.5'. This will be used to scale the Map component's link: Step to 50%
4343
const [selectedNode, setSelectedNode] = useState('root'); // We create a local state "selectedNode" and set it to a string 'root'.
4444
const [, dispatch] = useStoreContext(); // we destructure the returned context object from the invocation of the useStoreContext function to get access to our dispatch function
4545

src/app/components/StateRoute/ComponentMap/LinkControls.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ const collectNodes = (node: Node): void => {
3838
export default function LinkControls({
3939
layout, // from the layout local state (initially 'cartesian') in 'ComponentMap'
4040
linkType, // from linkType local state (initially 'vertical') in 'ComponentMap'
41-
stepPercent, // from stepPercent local state (initially '10') in 'ComponentMap'
41+
stepPercent, // from stepPercent local state (initially '0.5') in 'ComponentMap'
4242
setLayout, // from the layout local state in 'ComponentMap'
4343
setOrientation, // from the orientation local state in 'ComponentMap'
4444
setLinkType, // from the linkType local state in 'ComponentMap'
@@ -63,7 +63,8 @@ export default function LinkControls({
6363
</select>
6464
&nbsp;&nbsp;
6565

66-
<label>Orientation:</label> {/* Controls for the Orientation selection, this dropdown will be disabled when the polar layout is selected as it is not needed */}
66+
<label>Orientation:</label> {/* Toggle record button to pause state changes on target application */}
67+
{/* Controls for the Orientation selection, this dropdown will be disabled when the polar layout is selected as it is not needed */}
6768
&nbsp;
6869
<select
6970
onClick={(e) => e.stopPropagation()}

src/app/components/StateRoute/History.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ const defaultMargin: DefaultMargin = {
2424
// below we destructure the props
2525
function History(props: Record<string, unknown>): JSX.Element {
2626
const {
27-
width: totalWidth,
28-
height: totalHeight,
27+
width: totalWidth, // from ParentSize provided in StateRoute
28+
height: totalHeight, // from ParentSize provided in StateRoute
2929
margin = defaultMargin,
3030
hierarchy, // from 'tabs[currentTab]' object in 'MainContainer'
3131
dispatch, // from useStoreContext in 'StateRoute'

src/app/components/StateRoute/PerformanceVisx/BarGraph.tsx

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -54,26 +54,28 @@ const BarGraph = (props: BarGraphProps): JSX.Element => {
5454
showTooltip // function to set tooltip state
5555
} = useTooltip<TooltipData>(); // returns an object with several properties that you can use to manage the tooltip state of your component
5656
let tooltipTimeout: number;
57-
const {
57+
const {
5858
containerRef, // Access to the container's bounding box. This will be empty on first render.
5959
TooltipInPortal // TooltipWithBounds in a Portal, outside of your component DOM tree
60-
} = useTooltipInPortal({ // Visx hook
60+
} = useTooltipInPortal({ // Visx hook
6161
detectBounds: true, // use TooltipWithBounds
6262
scroll: true, // when tooltip containers are scrolled, this will correctly update the Tooltip position
6363
});
6464

6565
const keys = Object.keys(data.componentData);
6666
const getSnapshotId = (d: snapshot) => d.snapshotId; // data accessor (used to generate scales) and formatter (add units for on hover box). d comes from data.barstack post filtered data
6767

68+
const getSnapshotId = (d: snapshot) => d.snapshotId; // data accessor (used to generate scales) and formatter (add units for on hover box). d comes from data.barstack post filtered data
6869
const formatSnapshotId = (id) => `Snapshot ID: ${id}`; // returns snapshot id when invoked in tooltip section
6970
const formatRenderTime = (time) => `${time} ms `; // returns render time when invoked in tooltip section
7071

72+
7173
const snapshotIdScale = scaleBand<string>({ // create visualization SCALES with cleaned data
7274
domain: data.barStack.map(getSnapshotId),
7375
padding: 0.2,
7476
});
7577

76-
const renderingScale = scaleLinear<number>({ // Adjusts y axis to match/ bar height
78+
const renderingScale = scaleLinear<number>({ // Adjusts y axis to match/ bar height
7779
domain: [0, data.maxTotalRender],
7880
nice: true,
7981
});
@@ -94,7 +96,7 @@ const BarGraph = (props: BarGraphProps): JSX.Element => {
9496
title: tabs[currentTab].title,
9597
data,
9698
};
97-
99+
98100
useEffect(() => { // Animates the save series button.
99101
const saveButtons = document.getElementsByClassName('save-series-button'); // finds the buttom in the DOM
100102
for (let i = 0; i < saveButtons.length; i++) {
@@ -108,19 +110,20 @@ const BarGraph = (props: BarGraphProps): JSX.Element => {
108110
}
109111
});
110112

111-
const saveSeriesClickHandler = () => { // CURRENTLY DOES NOT SAVE
113+
const saveSeriesClickHandler = () => { // function to save the currently selected series
112114
if (tabs[currentTab].seriesSavedStatus === 'inputBoxOpen') {
113115
const actionNames = document.getElementsByClassName('actionname');
114116
for (let i = 0; i < actionNames.length; i += 1) {
115117
toStorage.data.barStack[i].name = actionNames[i].value;
116118
}
117-
dispatch(save(toStorage, seriesNameInput));
118-
setSeriesNameInput(`Series ${comparison.length}`);
119+
dispatch(save(toStorage, seriesNameInput)); // saves the series under seriesName
120+
setSeriesNameInput(`Series ${comparison.length}`); // sends a reducer that saves the series/toStorage object the user wants to chrome local storage
119121
return;
120122
}
121123
dispatch(save(toStorage)); // sends a reducer that saves the series/toStorage object the user wants to chrome local storage
122124
};
123125

126+
124127
const textbox = // Need to change so textbox isn't empty before saving
125128
tabs[currentTab].seriesSavedStatus === 'inputBoxOpen' ? (
126129
<input
@@ -215,7 +218,6 @@ const BarGraph = (props: BarGraphProps): JSX.Element => {
215218
}, 300)),
216219
);
217220
}}
218-
219221
onMouseMove={(event) => { // Cursor position in window updates position of the tool tip.
220222
dispatch(onHover(data.componentData[bar.key].rtid));
221223
if (tooltipTimeout) clearTimeout(tooltipTimeout);
@@ -294,8 +296,8 @@ const BarGraph = (props: BarGraphProps): JSX.Element => {
294296
{' '}
295297
<strong>{tooltipData.key}</strong>{' '}
296298
</div>
297-
<div>{data.componentData[tooltipData.key].stateType}</div>
298-
<div> {formatRenderTime(tooltipData.bar.data[tooltipData.key])} </div>
299+
<div>{'State: ' + data.componentData[tooltipData.key].stateType}</div>
300+
<div> {'Render time: ' + formatRenderTime(tooltipData.bar.data[tooltipData.key])} </div>
299301
<div>
300302
{' '}
301303
<small>{formatSnapshotId(getSnapshotId(tooltipData.bar.data))}</small>

0 commit comments

Comments
 (0)