thegrumpys / odop

Open Design Optimization Platform (ODOP) - Coil spring design app; mechanical springs; compression spring, extension spring, torsion spring
https://www.springdesignsoftware.org
MIT License
4 stars 5 forks source link

<p> cannot appear as descendant of <p> #741

Closed grumpyinca closed 2 years ago

grumpyinca commented 2 years ago

Red traceback message in Developer tools console appears when running the tutor8 script on the p6 --> 7 transition. It is not necessary to interact with the script. Just keep pressing "Next".

index.js:1437

   Warning: validateDOMNesting(...): <p> cannot appear as a descendant of <p>.
in p (at tutor8.js:289)
in p (at tutor8.js:288)
in div (at ExecutePanel.js:164)
in div (created by Row)
in Row (at ExecutePanel.js:163)
in div (created by Container)
in Container (at ExecutePanel.js:151)
in div (created by Alert)
in Transition (created by Fade)
in Fade (created by Alert)
in Alert (at ExecutePanel.js:150)
in ExecutePanel (created by Connect(ExecutePanel))
1fifoto commented 2 years ago

There are mismatched

tags which need to be changed to be matched

. Change tutor8.js from

                <p>
                <p>
                Based on the Objective Value, it appears that moving the constraint from
                its original value of 2.0 inches to a new value of approximately 2.6 inches 
                produces a result that is is right at the edge of feasibility.
                </p>

                Of course it is possible to combine any or all of these approaches.
                Trade will be happy to calculate how far it is necessary to move
                if multiple constraints are to be adjusted concurrently ... 
                just give more than one non-zero entry
                when defining the weights at the beginning of Trade's process.
                </p>

to

                <p>
                Based on the Objective Value, it appears that moving the constraint from
                its original value of 2.0 inches to a new value of approximately 2.6 inches 
                produces a result that is is right at the edge of feasibility.
                </p>

                <p>
                Of course it is possible to combine any or all of these approaches.
                Trade will be happy to calculate how far it is necessary to move
                if multiple constraints are to be adjusted concurrently ... 
                just give more than one non-zero entry
                when defining the weights at the beginning of Trade's process.
                </p>

Merged into master, closing