Skip to content

Conversation

Rajdeepc
Copy link
Contributor

@Rajdeepc Rajdeepc commented Mar 17, 2025

Description

Combobox fast follows fixes.

Motivation and context

  • SWC-582

How has this been tested?

  • Test case 1

    1. Go here
  • Did it pass in Desktop?

  • Did it pass in Mobile?

  • Did it pass in iPad?

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Checklist

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • If my change required a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have reviewed at the Accessibility Practices for this feature, see: Aria Practices

Best practices

This repository uses conventional commit syntax for each commit message; note that the GitHub UI does not use this by default so be cautious when accepting suggested changes. Avoid the "Update branch" button on the pull request and opt instead for rebasing your branch against main.

@Rajdeepc Rajdeepc self-assigned this Mar 17, 2025
@Rajdeepc Rajdeepc requested a review from a team as a code owner March 17, 2025 07:35
Copy link

changeset-bot bot commented Mar 17, 2025

🦋 Changeset detected

Latest commit: 275275e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 84 packages
Name Type
@spectrum-web-components/picker-button Minor
@spectrum-web-components/styles Minor
@spectrum-web-components/combobox Minor
@spectrum-web-components/bundle Minor
@spectrum-web-components/card Minor
@spectrum-web-components/illustrated-message Minor
@spectrum-web-components/custom-vars-viewer Minor
example-project-rollup Patch
example-project-webpack Patch
@spectrum-web-components/vrt-compare Minor
@spectrum-web-components/theme Minor
@spectrum-web-components/truncated Minor
documentation Patch
@spectrum-web-components/overlay Minor
@spectrum-web-components/slider Minor
@spectrum-web-components/story-decorator Minor
@spectrum-web-components/contextual-help Minor
@spectrum-web-components/menu Minor
@spectrum-web-components/picker Minor
@spectrum-web-components/popover Minor
@spectrum-web-components/tooltip Minor
@spectrum-web-components/breadcrumbs Minor
@spectrum-web-components/action-menu Minor
@spectrum-web-components/action-bar Minor
@spectrum-web-components/eslint-plugin Minor
@spectrum-web-components/accordion Minor
@spectrum-web-components/action-button Minor
@spectrum-web-components/action-group Minor
@spectrum-web-components/alert-banner Minor
@spectrum-web-components/alert-dialog Minor
@spectrum-web-components/asset Minor
@spectrum-web-components/avatar Minor
@spectrum-web-components/badge Minor
@spectrum-web-components/button-group Minor
@spectrum-web-components/button Minor
@spectrum-web-components/checkbox Minor
@spectrum-web-components/clear-button Minor
@spectrum-web-components/close-button Minor
@spectrum-web-components/coachmark Minor
@spectrum-web-components/color-area Minor
@spectrum-web-components/color-field Minor
@spectrum-web-components/color-handle Minor
@spectrum-web-components/color-loupe Minor
@spectrum-web-components/color-slider Minor
@spectrum-web-components/color-wheel Minor
@spectrum-web-components/dialog Minor
@spectrum-web-components/divider Minor
@spectrum-web-components/dropzone Minor
@spectrum-web-components/field-group Minor
@spectrum-web-components/field-label Minor
@spectrum-web-components/help-text Minor
@spectrum-web-components/icon Minor
@spectrum-web-components/icons-ui Minor
@spectrum-web-components/icons-workflow Minor
@spectrum-web-components/icons Minor
@spectrum-web-components/iconset Minor
@spectrum-web-components/infield-button Minor
@spectrum-web-components/link Minor
@spectrum-web-components/meter Minor
@spectrum-web-components/modal Minor
@spectrum-web-components/number-field Minor
@spectrum-web-components/progress-bar Minor
@spectrum-web-components/progress-circle Minor
@spectrum-web-components/radio Minor
@spectrum-web-components/search Minor
@spectrum-web-components/sidenav Minor
@spectrum-web-components/split-view Minor
@spectrum-web-components/status-light Minor
@spectrum-web-components/swatch Minor
@spectrum-web-components/switch Minor
@spectrum-web-components/table Minor
@spectrum-web-components/tabs Minor
@spectrum-web-components/tags Minor
@spectrum-web-components/textfield Minor
@spectrum-web-components/thumbnail Minor
@spectrum-web-components/toast Minor
@spectrum-web-components/top-nav Minor
@spectrum-web-components/tray Minor
@spectrum-web-components/underlay Minor
@spectrum-web-components/base Minor
@spectrum-web-components/grid Minor
@spectrum-web-components/opacity-checkerboard Minor
@spectrum-web-components/reactive-controllers Minor
@spectrum-web-components/shared Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Mar 17, 2025

Branch preview

Review the following VRT differences

When a visual regression test fails (or has previously failed while working on this branch), its results can be found in the following URLs:

If the changes are expected, update the current_golden_images_cache hash in the circleci config to accept the new images. Instructions are included in that file.
If the changes are unexpected, you can investigate the cause of the differences and update the code accordingly.

Copy link
Contributor

Tachometer results

Currently, no packages are changed by this PR...

@coveralls
Copy link
Collaborator

coveralls commented Mar 17, 2025

Pull Request Test Coverage Report for Build 14125197918

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 98.002%

Totals Coverage Status
Change from base Build 14125070024: 0.0%
Covered Lines: 33711
Relevant Lines: 34211

💛 - Coveralls

@marissahuysentruyt marissahuysentruyt force-pushed the rajdeep/combobox-fastfollows branch 2 times, most recently from c18b13f to 3998d91 Compare March 25, 2025 16:23
Rajdeep Chandra and others added 3 commits March 25, 2025 12:25
- corrects the s2 foundations disabled border color
- corrects the s2 foundations focus+hover border color
@marissahuysentruyt marissahuysentruyt force-pushed the rajdeep/combobox-fastfollows branch from 3998d91 to a0bcc97 Compare March 25, 2025 16:25
@marissahuysentruyt marissahuysentruyt self-assigned this Mar 25, 2025
marissahuysentruyt and others added 3 commits March 26, 2025 12:04
* fix(pickerbutton): s2 foundations fast follows

* chore(pickerbutton): create changeset

---------

Co-authored-by: Rajdeep Chandra <rajrock38@gmail.com>
@Rajdeepc Rajdeepc force-pushed the rajdeep/combobox-fastfollows branch from de82d8f to 60ab985 Compare March 27, 2025 10:28
@Rajdeepc Rajdeepc enabled auto-merge (squash) March 28, 2025 09:40
@Rajdeepc Rajdeepc disabled auto-merge March 28, 2025 09:40
@Rajdeepc Rajdeepc merged commit fa4be70 into main Mar 28, 2025
24 of 26 checks passed
@Rajdeepc Rajdeepc deleted the rajdeep/combobox-fastfollows branch March 28, 2025 09:40
castastrophe pushed a commit that referenced this pull request Apr 1, 2025
* chore(combobox): s2 fast follows fix

* ci: updated golden image cache

* chore: updated textfield and pickerbutton to latest to support s2 combobox css updates

* chore: updated tokens

* chore: upgraded picker button to its latest

* chore(combobox): s2 fast follows fix

* ci: updated golden image cache

* fix(combobox): design feedback updates

- corrects the s2 foundations disabled border color
- corrects the s2 foundations focus+hover border color

* chore(combobox): create changeset

* fix(pickerbutton): picker button version update (#5252)

* fix(pickerbutton): s2 foundations fast follows

* chore(pickerbutton): create changeset

---------

Co-authored-by: Rajdeep Chandra <rajrock38@gmail.com>

* ci: updated golden image cache

* chore: updated yarn lock

* chore: css changes

* ci: updated golden image cache

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@rajdeeps-mbp-2.macromedia.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>
Co-authored-by: Marissa Huysentruyt <marissa.huysentruyt@heysparkbox.com>
Co-authored-by: Marissa Huysentruyt <69602589+marissahuysentruyt@users.noreply.github.com>
Co-authored-by: Casey Eickhoff <48574582+caseyisonit@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants