Skip to content

Commit

Permalink
flesh out more of pdomOrder for Continuous screen, #53
Browse files Browse the repository at this point in the history
  • Loading branch information
pixelzoom committed Jun 7, 2021
1 parent ded435f commit c0888b1
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 2 deletions.
5 changes: 5 additions & 0 deletions js/continuous/view/ComponentsChartNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,11 @@ class ComponentsChartNode extends Node {

// @public for layout
this.chartRectangle = chartRectangle;

// pdom - traversal order
this.pdomOrder = [
//TODO x-axis zoom button group
];
}
}

Expand Down
5 changes: 5 additions & 0 deletions js/continuous/view/ContinuousAmplitudesChartNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,11 @@ class ContinuousAmplitudesChartNode extends Node {

// @public for layout
this.chartRectangle = chartRectangle;

// pdom - traversal order
this.pdomOrder = [
continuousWaveformCheckbox
];
}
}

Expand Down
5 changes: 3 additions & 2 deletions js/continuous/view/ContinuousScreenView.js
Original file line number Diff line number Diff line change
Expand Up @@ -213,10 +213,11 @@ class ContinuousScreenView extends ScreenView {
// See https://github.com/phetsims/fourier-making-waves/issues/53
screenViewRootNode.pdomOrder = [
controlPanel,
amplitudesChartNode,
componentsExpandCollapseButton,
//TODO put componentsChartNode here
componentsChartNode,
sumExpandCollapseButton,
//TODO put sumChartNode here
sumChartNode,
//TODO put width indicators here
resetAllButton
];
Expand Down
6 changes: 6 additions & 0 deletions js/continuous/view/ContinuousSumChartNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,12 @@ class ContinuousSumChartNode extends Node {

// @public for layout
this.chartRectangle = chartRectangle;

// pdom - traversal order
this.pdomOrder = [
//TODO x-axis zoom button group
envelopeCheckbox
];
}
}

Expand Down

0 comments on commit c0888b1

Please sign in to comment.