Closed kswenson closed 1 month ago
Attention: Patch coverage is 97.77778%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 85.81%. Comparing base (
e753e43
) to head (e8445d2
). Report is 2 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Project | codap-v3 |
Status | Passed |
Commit | 696ce01760 |
Started | Aug 9, 2024 6:15 AM |
Ended | Aug 9, 2024 6:24 AM |
Duration | 09:02 💡 |
OS | Linux Ubuntu - |
Browser | Chrome 127 ![]() |
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud
[PT-181840926]
Adds basic text component functionality:
dashboard
to facilitate testingStill to do:
Some independent changes in service of this PR were submitted separately as #1389, #1390. This PR is rooted on those and should not be merged until those have been merged.
Along the way, several utilities were developed that may have use elsewhere. Some of them were developed for use in positioning the toolbar alongside the component but I ended up solving that problem another way, so in some cases these utilities are not used in this PR but I'm checking them in anyway on the grounds that they seem likely to be useful at some point.
useCodapComponentContext
-- provides access to the.codap-component
via React context. There are a number of places in the code where we use DOM mechanisms to find the.codap-component
. This custom hook provides a ref via context which should be preferable.useComponentWrapperContext
-- provides access to the.free-tile-component
(parent of the.codap-component
) using the same mechanism.useDocumentContainerContext
-- provides access to the.codap-container
using the same mechanism.useMutationObserver
-- custom hook to provide access to DOM mutation observer functionality.