spike on allowing text #4659
Merged
spike on allowing text #4659
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
|
khiga8
force-pushed
the
spike-if-allowing-text
branch
from
bc1a30a
to
ce69bc9
Compare
size-limit report 📦
|
TylerJDev
approved these changes
Jun 10, 2024
TylerJDev
merged commit 804153b
into
v-team-trailing-action-action-list
24 of 27 checks passed
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jun 21, 2024
* Add fixes for ActionList * More type fixes * temp type fixes * Add condition for inactive items * add yet another condition * chore(deps): bump changesets/action from 1.4.5 to 1.4.6 (#4282) Bumps [changesets/action](https://github.com/changesets/action) from 1.4.5 to 1.4.6. - [Release notes](https://github.com/changesets/action/releases) - [Changelog](https://github.com/changesets/action/blob/main/CHANGELOG.md) - [Commits](changesets/action@f13b1ba...e2f8e96) --- updated-dependencies: - dependency-name: changesets/action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test(e2e): add e2e test for SelectPanel2 default story (#4279) * fix(SelectPanel2): add aria-labelledby to listbox * test(e2e): add e2e test for SelectPanel2 default story * chore: add changeset * test(vrt): update snapshots --------- Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * Address a few v8 color bugs (#4278) * small bug fixes with v8 * Create khaki-schools-lay.md * test(vrt): update snapshots * snippy snaps * test(vrt): update snapshots * test: update snapshots * test: update snapshots * try commenting flakey tests * test: comment out flakey snapshot test --------- Co-authored-by: langermank <langermank@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * chore(deps-dev): bump ip from 2.0.0 to 2.0.1 (#4291) Bumps [ip](https://github.com/indutny/node-ip) from 2.0.0 to 2.0.1. - [Commits](indutny/node-ip@v2.0.0...v2.0.1) --- updated-dependencies: - dependency-name: ip dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Change how `ref` is handled * Change types * Update storybook example types * Update types on component * Add another type * Update type in `SegmentedControl` * Add back `li`-only type * Add onto `onSelect` type * Update more types * Type fixes for `LinkItem` * Changes from feedback * Change types * Replace `role="list"` with context * Add feature flag to `ActionList.Item * Add back forwardedRef in cases where valid role is true * Update FF name * Add lint disable * Update FF name * Add changeset * Remove `list` condition * Rename FF * Address feedback * Add feature flag story * Add new test * Add export * Updated forwardRef to be polymorphic * Spike: Add trailingAction to ActionList.Item Co-authored-by: Tyler Jones <tylerjdev@github.com> * Updated forwardRef to be polymorphic * TrailingAction style adjustments * test(vrt): update snapshots * Remove unused story * Limit TrailingAction props * Fix missing href for as link * Add Inactive example * Add styles for `showOnHover` * Fixed href overload issue Co-authored-by: Tyler Jones <tylerjdev@github.com> * Made sure that TrailingAction doesn't show when inactive state Co-authored-by: Tyler Jones <tylerjdev@github.com> * test(vrt): update snapshots * Add fixes for button styles * Fix for flex styles * Update storybookd descriptions * Add block description example * Add tests * Remove FF from tests * update e2e stories * spike on allowing text (#4659) * spike on allowing text * showOnHover support * Add overflow example * Add visual tests * Update from display to visibility * Create calm-crabs-raise.md * Disallow usage in `ActionMenu` * Update ActionList.test.tsx * Remove hover/focus styles * test(vrt): update snapshots * Apply suggestions from code review * Apply suggestions from code review * Fix merge conflict * Remove Hover examples * Remove showOnHover code * Add FF const back * Remove some usage of `hoverStyles` * test(vrt): update snapshots * Add props to TrailingAction * update types Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * update docs --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Tyler Jones <tylerjdev@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Katie Langerman <18661030+langermank@users.noreply.github.com> Co-authored-by: langermank <langermank@users.noreply.github.com> Co-authored-by: khiga8 <khiga8@users.noreply.github.com> Co-authored-by: TylerJDev <TylerJDev@users.noreply.github.com>
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.
TrailingActionwill render as anIconButton variant="invisible"orButton variant="invisible"depending on whether or noticonis passed intoTrailingAction.