fix: add start end suffix for callout and add eslint-config #17
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes Jira WARP-527
Add @warp-ds/eslint-config as a devDependency
Add linting scripts to package.json
Lint all code using
pnpm lint
Add github workflow for linting PR.
Remove
middlePosition
,isDirectionVertical
andcomputeCalloutArrow()
and compute callout's arrow position in thecomputePosition()
instead.Add a check to only apply
left
andtop
style toattentionEl
whenisCallout
isfalse
, since we only want to use Floating-ui'scomputePosition()
to calculate the callout's arrow position and not the attention element's position, due to thattargetEl
can be undefined whenisCallout
istrue
.How to test:
fix/add-start-end-suffix-callout
in @warp-ds/react, @warp-ds/vue and/or @warp-ds/elements