-
Notifications
You must be signed in to change notification settings - Fork 5
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(shoelace): tests failed to check hidden menu #237
Conversation
🦋 Changeset detectedLatest commit: 8e4e1ea The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
Codecov Report
@@ Coverage Diff @@
## master #237 +/- ##
==========================================
+ Coverage 78.00% 78.02% +0.01%
==========================================
Files 166 166
Lines 7635 7641 +6
Branches 587 588 +1
==========================================
+ Hits 5956 5962 +6
Misses 1653 1653
Partials 26 26
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -20,7 +25,7 @@ export class ShSlAutocomplete extends LitElement { | |||
@property({ type: String }) | |||
inputValue = '' | |||
|
|||
@property({ type: Boolean, reflect: true }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is reflect?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Reflect is to keep a sync between HTML element attribute and property value.
https://lit.dev/docs/components/properties/#reflected-attributes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok what I saw.
Refactor and fix a brittle test which failed #236
Apparently it's a bad idea to have a reflected boolean attribute set to false