metalabdesign / flowtip

88%

master: 57%

LAST BUILD ON BRANCH calculate-edge-offset

branch: calculate-edge-offset

CHANGE BRANCH

x

Reset

• calculate-edge-offset
• 4.x.x
• 5.x.x
• bump-version
• comment-typos
• container-detection
• decoupled-render
• disable-travis-email
• es-modules
• fiber-support
• fix-incorrect-edge-detection
• fix-incorrect-rotation
• fix-readme-css
• fix-scroll-jank
• fix-scrollers
• fix-things
• fix-typo-bug
• flow-lint
• git-links
• master
• more-updates
• prettier
• remove-fbjs
• remove-old-node
• remove-omit-util
• renovate/babel-monorepo
• renovate/babel-plugin-module-resolver-3.x
• renovate/configure
• renovate/enzyme-3.x
• renovate/enzyme-adapter-react-16-1.x
• renovate/eslint-4.x
• renovate/eslint-5.x
• renovate/eslint-config-metalab-10.x
• renovate/eslint-plugin-prettier-2.x
• renovate/eslint-plugin-prettier-3.x
• renovate/flowgen-1.x
• renovate/jest-monorepo
• renovate/lerna-2.x
• renovate/lerna-3.x
• renovate/lerna-monorepo
• renovate/major-jest-monorepo
• renovate/pin-dependencies
• renovate/prettier-1.x
• renovate/react-16.x
• renovate/react-dom-16.x
• renovate/react-monorepo
• renovate/regenerator-runtime-0.x
• renovate/renamer-0.x
• renovate/rimraf-2.x
• renovate/rollup-1.x
• renovate/rollup-plugin-babel-4.x
• renovate/rollup-plugin-commonjs-9.x
• renovate/rollup-plugin-node-resolve-4.x
• renovate/typescript-3.x
• standardize-comments
• static-site
• storybook
• tests
• typescript
• update-eslint-babel
• update-react-versions
• update-tests
• updates
• upgrade-deps
• v5.0.0
• v5.1.0
• v5.2.0
• v5.2.1
• x

Build # 258

Build Type

push

travis-ci

Committed by 10xjs

Commit Message

Include edgeOffset in layout calculation

`edgeOffset` is used to represent a margin at the the bounds rect edge. Previously it was only a prop used within the `FlowTip` react component that was subtracted from the resolved `bounds` rect, effectively shrinking it. This had the undesirable effect of also applying the margin to between the positioned tooltip and viewport edge when the target was outside the viewport.

What we really want the tail side of the tooltip to have its own margin, separate from the body. When the tooltip is positioned at the screen edge, the larger of the two margins will determine the final position.

Run Details

429 of 481 branches covered (89.19%)

347 of 395 relevant lines covered (87.85%)

44.62 hits per line