Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(sbb-option, sbb-optgroup, sbb-autocomplete, sbb-select): lit migration #2062

Merged
merged 18 commits into from
Oct 13, 2023

Conversation

TomMenga
Copy link
Contributor

No description provided.

@TomMenga TomMenga force-pushed the migr/sbb-autocomplete branch 2 times, most recently from 582e048 to 4e99d50 Compare October 10, 2023 10:55
@github-actions github-actions bot temporarily deployed to preview-pr2062 October 10, 2023 11:02 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Oct 10, 2023

Codecov Report

❗ No coverage uploaded for pull request base (lit-migration@dc981b5). Click here to learn what that means.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@               Coverage Diff                @@
##             lit-migration    #2062   +/-   ##
================================================
  Coverage                 ?   92.07%           
================================================
  Files                    ?      161           
  Lines                    ?    22655           
  Branches                 ?     1132           
================================================
  Hits                     ?    20860           
  Misses                   ?     1782           
  Partials                 ?       13           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@jeripeierSBB jeripeierSBB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only partial review, stopped at sbb-optgroup

<sbb-form-field>
<input />
<sbb-autocomplete id="myAutocomplete">
<sbb-autocomplete id="myAutocomplete" style="--sbb-options-panel-animation-duration: 0.01s">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about using disableAnimation property? this would be the public API of the autocomplete

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was the first choice but, for reasons I haven't been able to explain, if I do that the animationend event is not fired. After half a day of debugging, I surrendered and used that workaround.
I accepted the compromise since the true meaning of that is just to speed up tests to avoid the timeout (since they have a lot of close/open interactions)

src/components/sbb-autocomplete/sbb-autocomplete.e2e.ts Outdated Show resolved Hide resolved
src/components/sbb-autocomplete/sbb-autocomplete.e2e.ts Outdated Show resolved Hide resolved
src/components/sbb-autocomplete/sbb-autocomplete.tsx Outdated Show resolved Hide resolved
src/components/sbb-autocomplete/sbb-autocomplete.tsx Outdated Show resolved Hide resolved
@TomMenga TomMenga force-pushed the migr/sbb-autocomplete branch from 4e99d50 to 3675c60 Compare October 10, 2023 13:12
src/components/sbb-optgroup/sbb-optgroup.tsx Outdated Show resolved Hide resolved
src/components/sbb-option/sbb-option.tsx Outdated Show resolved Hide resolved
src/components/sbb-select/sbb-select.tsx Outdated Show resolved Hide resolved
src/components/sbb-select/sbb-select.tsx Outdated Show resolved Hide resolved
src/components/sbb-select/sbb-select.tsx Outdated Show resolved Hide resolved
src/components/sbb-select/sbb-select.tsx Outdated Show resolved Hide resolved
@TomMenga TomMenga force-pushed the migr/sbb-autocomplete branch from 81ae599 to e222081 Compare October 11, 2023 15:29
@TomMenga TomMenga merged commit fcb5eac into lit-migration Oct 13, 2023
9 of 10 checks passed
@TomMenga TomMenga deleted the migr/sbb-autocomplete branch October 13, 2023 06:52
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.

3 participants