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

Fixes element measuring relative to fullbleed (better) #1558

Merged
merged 3 commits into from
May 12, 2020

Conversation

wassgha
Copy link
Contributor

@wassgha wassgha commented May 5, 2020

Closes #1545

@codecov
Copy link

codecov bot commented May 5, 2020

Codecov Report

Merging #1558 into master will increase coverage by 0.01%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1558      +/-   ##
============================================
+ Coverage     61.93%   61.94%   +0.01%     
  Complexity      216      216              
============================================
  Files           571      571              
  Lines          9191     9195       +4     
============================================
+ Hits           5692     5696       +4     
  Misses         3499     3499              
Impacted Files Coverage Δ Complexity Δ
...-story/components/canvas/canvasUploadDropTarget.js 50.00% <ø> (ø) 0.00 <0.00> (ø)
...s/src/edit-story/components/canvas/displayLayer.js 14.28% <ø> (ø) 0.00 <0.00> (ø)
...ts/src/edit-story/components/canvas/framesLayer.js 23.07% <0.00%> (ø) 0.00 <0.00> (ø)
...rc/edit-story/components/canvas/selectionCanvas.js 6.25% <0.00%> (ø) 0.00 <0.00> (ø)
...src/edit-story/components/canvas/canvasProvider.js 76.92% <50.00%> (-1.46%) 0.00 <0.00> (ø)
assets/src/edit-story/components/canvas/layout.js 67.50% <66.66%> (+2.50%) 0.00 <0.00> (ø)
assets/src/edit-story/units/dimensions.js 100.00% <100.00%> (ø) 0.00 <0.00> (ø)

@github-actions
Copy link
Contributor

github-actions bot commented May 5, 2020

Size Change: +212 B (0%)

Total Size: 853 kB

Filename Size Change
assets/js/edit-story.js 405 kB +130 B (0%)
assets/js/stories-dashboard.js 441 kB +82 B (0%)
ℹ️ View Unchanged
Filename Size Change
assets/css/edit-story.css 3.04 kB 0 B
assets/css/stories-dashboard.css 3.04 kB 0 B

compressed-size-action

@wassgha wassgha requested a review from dvoytenko May 11, 2020 17:08
@wassgha wassgha marked this pull request as ready for review May 11, 2020 17:08
@wassgha
Copy link
Contributor Author

wassgha commented May 11, 2020

@dvoytenko ready :)

@wassgha wassgha merged commit 4a2391c into master May 12, 2020
@wassgha wassgha deleted the fix/fullbleed-lasso branch May 12, 2020 03:39
obetomuniz added a commit that referenced this pull request May 13, 2020
* master: (99 commits)
  Bump eslint-plugin-react from 7.19.0 to 7.20.0 (#1749)
  Checked if date is moment before wrapping
  Refactored formatted date tests
  Add spy to standardize timing on tests
  Remove tests that used to work in CI but no longer do
  Remove unused import
  Fix snapshot test for useTemplateAPI
  Added support for modal-routing on dashboard.
  Add additional sorting value to ensure the same order. (#1737)
  update tests around reshaping story object
  [ImgBot] Optimize images (#1723)
  Bump @testing-library/dom from 7.5.1 to 7.5.2 (#1732)
  Fixes element measuring relative to fullbleed (better) (#1558)
  swapping to use the decoded html version of the title we have access to to turn html symbols from &amp; to characters &
  adding check for user name to display author
  moving required for users prop type to component level so that we can reuse the shape without it being required. updating displayDate propType to object not string
  some organization of imports
  moving date formatting to the card title level and memoizing
  clean up translation for modified date string in card item
  no need to call edit function inline - this has no functional difference but still
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Group: Canvas Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lasso selection coordinates inaccurate
4 participants