fix: textarea: ensure grip icon doesn't block mouse drag #698
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit f76eec1:
|
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #698 +/- ##
==========================================
- Coverage 85.15% 85.14% -0.02%
==========================================
Files 881 881
Lines 17971 17971
Branches 6460 6460
==========================================
- Hits 15304 15302 -2
- Misses 2595 2597 +2
Partials 72 72 ☔ View full report in Codecov by Sentry. |
jhoward-eightfold
approved these changes
Aug 24, 2023
dkilgore-eightfold
added a commit
that referenced
this pull request
Sep 6, 2023
* fix: pagination: Issues with VisiblePagerCountSize (#610) * chore: systemuibutton: pass transparent down via props (#611) * chore(release): 2.36.0 * fix: select: ensure aria attributes target the intended element (#612) * fix: select: ensure aria attributes target the intended element * chore: dropdown: add ariaref ut and refine role null check * chore: dropdown: update role null check to handle all elements * chore(release): 2.36.1 * fix: stepper: handle scrollIntoView for both layouts on mount (#613) * chore(release): 2.36.2 * fix: button: pass disruptive down from props to button from defaultbutton (#614) * fix: dropdown: tweak css slideupin animation (#618) * fix: dropdown: tweak css slideupin animation * chore: dropdown: tweak animation curve and revert previous adjustment * fix: locale: add support for hi locale (#602) * fix: dropdown: fix visible prop (#616) * fix: dropdown: fix visible prop * chore: dropdown: update ut to open dropdown again * feat: empty: enable themes and adds two additional modes (#619) * chore: css: organize and annotate global variables (#621) * chore: css: organize and annotate global variables * chore: css: remove unused base color sass variables * chore(release): 2.37.0 * feat: messagebar: add message bar component (#622) * feat: messagebar: add message bar component * chore: messagebar: update icons per spec * chore: messagebar: export component * chore: messagebar: address pr feedback by adding a mapping * fix: css: global css variable name alignment with design specification (#625) * feat: rename background-color to background (#628) * feat: rename background-color to background * Use background for button css so that we can have more flexible customizations * fix: remove changes * remove changes * fix: table: fix loading compare and updates default spinner size and position (#627) * fix: table: fix loading compare and updates default spinner size and position * chore: table�: inherit spinner div height * fix: pill: ensure pill heights match latest design specification (#626) * fix: pill: ensure pill heights match latest design specification * chore: pill: address pr feedback and move pill border radius to global css var * feat: table: adds column border and disable row bg hover color props (#624) * fix: select: ensure medium and large pills fit inside inputs of same size (#630) * chore(release): 2.38.0 * feat: icon: add icons to support kanban view (#632) * feat: icon: add icons to support integration system simplification (#634) * fix: slider: increase vertical hit target and handle segmented gap clicks (#635) * fix: slider: increase vertical hit target and handle segmented gap clicks * fix: slider: ensure default behavior of tooltip does not hide on reference click * fix: select: Update selected option based on defaultValue when isLoading changes. (#631) Co-authored-by: Dylan <99700808+dkilgore-eightfold@users.noreply.github.com> * feat: icon: add collapse icon for kanban (#637) * chore: v2.38.2 hotfix fi to main (#638) * fix: dropdown: revert useonclickoutside to only toggle in specific conditions * chore(release): 2.38.1 * feat: custom fonts support via configProvider * fix: remove unnecessary import * chore(release): 2.38.2 * chore: semver: revert hotfix versioning ahead of fi --------- Co-authored-by: Jeff Howard <106610186+jhoward-eightfold@users.noreply.github.com> * fix: button: ensure loading dots use text color and enable two state loader (#636) * fix: button: ensure loading dots use text color and enable two state loader * chore: loader: address pr feedback by removing unneeded rtl code * chore(release): 2.39.0 * fix: select: revert update selected option based on defaultValue when isLoading (#643) * chore(release): 2.39.1 * fix: linkbutton: ensure default width matches min and max width (#640) * fix: matchscore: color contrast and vertical alignment fix up (#641) * fix: tabs: pill variant color contrast fix up (#642) * fix: table: various accessibility improvements (#644) * chore(release): 2.40.0 * fix: select: Update selected option based on defaultValue when isLoading changes (#646) * fix: picker: color contrast ratio fixes (#647) * fix: picker: color contrast ratio fixes * chore: picker: address pr feedback remove unused css vars and selectors * chore: storybook: update named export order to be more consistent using babel (#649) * chore: storybook: update named export order to be more consistent using babel * chore: storybook: removes redundant prop * chore(deps-dev): bump webpack from 5.64.4 to 5.76.0 (#564) Bumps [webpack](https://github.com/webpack/webpack) from 5.64.4 to 5.76.0. - [Release notes](https://github.com/webpack/webpack/releases) - [Commits](webpack/webpack@v5.64.4...v5.76.0) --- updated-dependencies: - dependency-name: webpack dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dylan <99700808+dkilgore-eightfold@users.noreply.github.com> * chore(deps-dev): bump semver from 7.3.5 to 7.5.2 (#650) Bumps [semver](https://github.com/npm/node-semver) from 7.3.5 to 7.5.2. - [Release notes](https://github.com/npm/node-semver/releases) - [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md) - [Commits](npm/node-semver@v7.3.5...v7.5.2) --- updated-dependencies: - dependency-name: semver dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: slider: ensure tooltip is visually hidden when slider is disabled (#651) * fix: menu: add htmltype prop to menu item button (#654) * fix: menu: add htmltype prop to menu item button * chore: menu: update snaps to account for new default property * feat: breadcrumb: add breadcrumb component (#629) * fix: slider: fixes hidetrack and hiderail props typos and ommissions (#656) * fix: slider: fixes hidetrack and hiderail props typos and ommissions * chore: slider: adds hidetrack and hiderail unit tests * chore: slider: swaps hidetrack and hiderail references of segments * fix: table: updates scrollable div by removing deprecated css property (#655) * fix: table: updates scrollable div by removing deprecated css property * chore: table: update snaps * chore: table: update border to hide in mozilla browsers * chore: table: updates snaps and sticky unit test * chore(release): 2.41.0 * feat: selectors: add var theme support to check box radio and toggle (#659) * feat: selectors: add var theme support to check box radio and toggle * chore: selectors: reverts transition change to limit scope * chore: selectors: address pr feedback * chore: dropdown: adds advanced visibility story and unit tests (#661) * chore: css: remove redundant font stack sass and use semantic variable (#660) * feat: persistentbar: support small screens and themes plus custom render (#667) * feat: persistentbar: support small screens and themes plus custom render * chore: persistentbar: address peer feedback and add css vars with border opt * chore: coverage: adds unit tests to components that need them (#670) * chore: coverage: adds unit tests to components that need them * chore: coverage: test for skeleton variant * fix: select: adds fall back and optional chaining to avoid null reference errors at runtime (#669) * fix: select: adds optional chaining to avoid null reference errors at runtime * chore: select: fall back to empty array rather than use optional chain * feat: slider and progress: add ability to visually hide borders (#673) * feat: slider: add ability to visually hide rail and track borders * feat: progress: enable discreet theming and border manipulation via props and css * chore(release): 2.42.0 * feat: infobar: adds bordered prop (#676) * feat: infobar: adds bordered prop * chore: infobar: align naming convention of css variables * chore(deps): bump tough-cookie from 4.0.0 to 4.1.3 (#664) Bumps [tough-cookie](https://github.com/salesforce/tough-cookie) from 4.0.0 to 4.1.3. - [Release notes](https://github.com/salesforce/tough-cookie/releases) - [Changelog](https://github.com/salesforce/tough-cookie/blob/master/CHANGELOG.md) - [Commits](salesforce/tough-cookie@v4.0.0...v4.1.3) --- updated-dependencies: - dependency-name: tough-cookie dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dylan <99700808+dkilgore-eightfold@users.noreply.github.com> * chore(deps): bump word-wrap from 1.2.3 to 1.2.4 (#671) Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4. - [Release notes](https://github.com/jonschlinkert/word-wrap/releases) - [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4) --- updated-dependencies: - dependency-name: word-wrap dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dylan <99700808+dkilgore-eightfold@users.noreply.github.com> * chore: documentation: updates readme and contributing markdown files (#677) * fix: table: adds default overflow back to latest scroll bars mixin (#678) * chore: codescan: create sonarqube yml * chore: codescan: update sonarqube yml * chore: codescan: update sonarqube yml secret * chore: codescan: update sonarqube yml args * chore: codescan: update sonarqube yml syntax * chore: codescan: create sonar project properties file * chore: codescan: update sonarqube yml defer to properties file * chore: codescan: move sonar properties file to root (#679) * chore: codescan: update sonarqube yml to use input rather than env vars * chore: codescan: revert sonarqube yml change and exclude github secret * chore: codescan: update sonarqube yml to use pull request target * chore: codescan: update sonarqube yml pull request type * chore: publish: update npm publish yml repo email alias * fix: accordion: fixes scroll bar animation flicker (#680) * feat: card: update card size prop and remove bunko story (#682) * chore(release): 2.43.0 * fix: popup: fixes style inheritance of reference element (#675) * fix: popup: fixes style inheritance of reference element * chore: popup: handle invalid react element * feat: badge: add badge size and update button and tab custom css (#672) * feat: badge: add badge size and update button and tab custom css * chore: coverage: adds more badge tests * chore: i18n: exports octuple locales and locale type (#684) * chore: i18n: exports octuple locales and locale type * chore: i18n: moves locale to separate entry * chore: i18n: apply structure * chore: governance: add security policy (#689) * fix: tooltip: fixes tooltip toggle on mobile devices (#668) * fix: tooltip: fixes tooltip toggle on mobile devices * chore: tooltip: adds click toggle unit test * chore: coverage: adds tooltip and mobile detector tests * chore: popup: adds istouching method to test for hybrid devices * chore: popup: update function name and ensure cloned element props are preserved * fix: button: moves button transition to global semantic variables (#695) * chore: icons: add new icon mdiAccountMultiplePlusOutline (#697) * feat: selectors: adds checkbox indeterminate state and selector pill variants (#694) * feat: selectors: adds checkbox indeterminate state and selector pill variants * chore: selectors: export types and add another checkbox unit test * fix: textarea: ensure grip icon doesn't block mouse drag (#698) * feat: pickers: changeonblur and button props with option to hide buttons (#696) * feat: pickers: changeonblur and button props with option to hide buttons * chore: pickers: adds unit tests and updates showtoday default * chore(release): 2.44.0 * fix: tooltip: fixes default clone element class names (#699) * fix: carousel: add controls prop to scroll menu (#705) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Ranjith Nori <106535945+rnori-eightfold@users.noreply.github.com> Co-authored-by: Yash raj chhabra <95337653+ychhabra-eightfold@users.noreply.github.com> Co-authored-by: Yash Raj Chhabra <ychhabra@eightfold.ai> Co-authored-by: dcoblentz-eightfold <119535119+dcoblentz-eightfold@users.noreply.github.com> Co-authored-by: Utsav Singhal <84313406+usinghal-eightfold@users.noreply.github.com> Co-authored-by: Aryan Mediratta <64970674+aryan-eightfold@users.noreply.github.com> Co-authored-by: Purushottam Tiwari <133845170+ptiwari-eightfold@users.noreply.github.com> Co-authored-by: Jeff Howard <106610186+jhoward-eightfold@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Abhishek Gupta <agupta@eightfold.ai>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
SUMMARY:
pointer-events: noneto grippy so it will not steal click and draggrippyFixup.mp4
GITHUB ISSUE (Open Source Contributors)
#693
JIRA TASK (Eightfold Employees Only):
ENG-62761
CHANGE TYPE:
TEST COVERAGE:
TEST PLAN:
Pull the PR branch and run
yarnandyarn storybook. Verify theText Areastory behaves as expected.