Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix positioning of sparrow buttons in Geometry #2435

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

bgoldowsky
Copy link
Contributor

@bgoldowsky bgoldowsky commented Nov 14, 2024

Fixes a regression introduced by the navigator panel in the Geometry tile.

The navigator-internal GeometryContentComponent was re-registering the tileAPI and returning incorrect position information for objects (based on where they are shown in the navigator view, rather than the main tile view).

Copy link

codecov bot commented Nov 14, 2024

Codecov Report

Attention: Patch coverage is 75.00000% with 18 lines in your changes missing coverage. Please review.

Project coverage is 85.38%. Comparing base (a6d5c58) to head (54d9403).
Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
src/components/tiles/geometry/geometry-content.tsx 75.00% 12 Missing and 6 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2435      +/-   ##
==========================================
- Coverage   86.13%   85.38%   -0.75%     
==========================================
  Files         746      746              
  Lines       38663    38664       +1     
  Branches     9871     9872       +1     
==========================================
- Hits        33301    33015     -286     
- Misses       5063     5345     +282     
- Partials      299      304       +5     
Flag Coverage Δ
cypress ?
cypress-regression 76.81% <75.00%> (-0.82%) ⬇️
cypress-smoke 28.47% <5.55%> (+<0.01%) ⬆️
jest 48.73% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Nov 14, 2024

collaborative-learning    Run #14150

Run Properties:  status check errored Errored #14150  •  git commit 54d94033ee: Don't re-register tile API in the geometry-content.tsx in the navigator.
Project collaborative-learning
Branch Review 188534586-fix-geometry-sparrows
Run status status check errored Errored #14150
Run duration 15m 40s
Commit git commit 54d94033ee: Don't re-register tile API in the geometry-content.tsx in the navigator.
Committer Boris Goldowsky
View all properties for this run ↗︎

Test results
Tests that failed  Failures 2
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 3
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 115
View all changes introduced in this branch ↗︎

@bgoldowsky bgoldowsky marked this pull request as ready for review November 14, 2024 22:04
@bgoldowsky
Copy link
Contributor Author

The regression test failure is just the one that's on master right now.

@bgoldowsky bgoldowsky requested a review from emcelroy November 14, 2024 22:05
Copy link
Contributor

@emcelroy emcelroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@bgoldowsky bgoldowsky merged commit a8cf9c7 into master Nov 15, 2024
15 of 19 checks passed
@bgoldowsky bgoldowsky deleted the 188534586-fix-geometry-sparrows branch November 15, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants