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: Remove small pixel differences in diagram cache dialog #1802

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

MoritzWeber0
Copy link
Member

Small pixel differences led to small jumps when the diagram cache was loaded. It feels smoother now.

Copy link

codecov bot commented Sep 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.18%. Comparing base (74f6b81) to head (87e4f37).
Report is 353 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1802   +/-   ##
=======================================
  Coverage   84.18%   84.18%           
=======================================
  Files         186      186           
  Lines        6159     6159           
  Branches      677      677           
=======================================
  Hits         5185     5185           
  Misses        826      826           
  Partials      148      148           

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

@MoritzWeber0 MoritzWeber0 marked this pull request as ready for review September 13, 2024 14:34
Copy link

github-actions bot commented Sep 13, 2024

A Storybook preview is available for commit 5934f2c.
➡️ View Storybook
➡️ View Chromatic build
✅ Captured 40 snapshots. No changes detected.

@MoritzWeber0 MoritzWeber0 changed the base branch from main to reg-actions September 14, 2024 13:58

This comment has been minimized.

@MoritzWeber0 MoritzWeber0 force-pushed the reg-actions branch 2 times, most recently from 20d2f1e to 300161c Compare September 14, 2024 14:36

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@MoritzWeber0 MoritzWeber0 force-pushed the reg-actions branch 2 times, most recently from e9bd352 to ce6c33a Compare September 17, 2024 15:23
Base automatically changed from reg-actions to main September 17, 2024 15:30
…adge

Small pixel differences led to small jumps when the diagram cache was
loaded. It feels smoother now.

Same applies to the model complexity badge.
Copy link

sonarcloud bot commented Sep 17, 2024

Copy link

This report was generated by comparing 5934f2c with 74f6b81.
If you would like to check difference, please check here.

change detected

ArtifactName: reg

item count
pass 228
change 14
new 2
delete 0
📝 Report

Differences

Model Components_Model Complexity Badge_Git Instance Endpoint Undefined.png

actual Actual
expected Expected
difference Difference

Model Components_Model Complexity Badge_Loading.png

actual Actual
expected Expected
difference Difference

Model Components_Model Complexity Badge_No Job Configured.png

actual Actual
expected Expected
difference Difference

Model Components_Model Complexity Badge_Pipeline Job Not Found.png

actual Actual
expected Expected
difference Difference

Model Components_Model Complexity Badge_Successful.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Combined_desktop.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Combined_mobile.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Diagram Error_desktop.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Diagram Error_mobile.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Diagram Not Loaded_desktop.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Diagram Not Loaded_mobile.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Loading_desktop.png

actual Actual
expected Expected
difference Difference

Model Components_Model Diagram Cache_Loading_mobile.png

actual Actual
expected Expected
difference Difference

Model Components_Model Overview_Overview_desktop.png

actual Actual
expected Expected
difference Difference

New Items

Model Components_Model Diagram Cache_Loading Without Scroll_desktop.png

NewItem

Model Components_Model Diagram Cache_Loading Without Scroll_mobile.png

NewItem

@MoritzWeber0 MoritzWeber0 merged commit e4b9f6b into main Sep 17, 2024
32 checks passed
@MoritzWeber0 MoritzWeber0 deleted the diagram-cache-ui branch September 17, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant