-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
bug(bottom-sheet): Config values throw error but work #28832
Closed
1 task
TheColorRed opened this issue
Apr 5, 2024
· 1 comment
· Fixed by #29794 or Mazen-Embaby/components#1
Closed
1 task
bug(bottom-sheet): Config values throw error but work #28832
TheColorRed opened this issue
Apr 5, 2024
· 1 comment
· Fixed by #29794 or Mazen-Embaby/components#1
Labels
Comments
crisbeto
added
P4
A relatively minor issue that is not relevant to core functions
area: material/bottom-sheet
and removed
needs triage
This issue needs to be triaged by the team
labels
Apr 14, 2024
naaajii
added a commit
to naaajii/components
that referenced
this issue
Sep 27, 2024
…onfig these properties were missing from the config but still worked as they were passed to dialog under the hood to avoid type errors fixes angular#28832
naaajii
added a commit
to naaajii/components
that referenced
this issue
Sep 27, 2024
…onfig these properties were missing from the config but still worked as they were passed to dialog under the hood to avoid type errors fixes angular#28832
naaajii
added a commit
to naaajii/components
that referenced
this issue
Sep 27, 2024
…onfig these properties were missing from the config but still worked as they were passed to dialog under the hood to avoid type errors fixes angular#28832
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Is this a regression?
The previous version in which this bug was not present was
No response
Description
When creating a bottom sheet, you cannot pass
height
,minHeight
,maxHeight
to the sheet as it throws an error:However, when used with
@ts-ignore
, the properties works as expected.Reproduction
StackBlitz link: https://stackblitz.com/edit/stackblitz-starters-fzwkec?file=src%2Fmain.ts
Steps to reproduce:
// @ts-ignore
lineExpected Behavior
These properties should be acceptable.
Actual Behavior
These properties throw errors unless
// @ts-ignore
is used.Environment
The text was updated successfully, but these errors were encountered: