-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Closed
Closed
Copy link
Labels
P3An issue that is relevant to core functions, but does not impede progress. Important, but not urgentAn issue that is relevant to core functions, but does not impede progress. Important, but not urgentarea: material/core
Description
Is this a regression?
- Yes, this behavior used to work in the previous version
The previous version in which this bug was not present was
No response
Description
get-theme
Reproduction
Stackblitz doesn't include newest version of Angular Material (^18), so can't be produced.
Basically, an invocation of get-theme-color($theme, primary, neutral, 90) throws an error because it does not handle the case in which get-theme-color receives four arguments.
It can be seen in the picture below that, even though it is clear a function can expect 4 arguments ($theme and $args, where length of $args can be 3), the case for $args-count == 3 is not handled.
Expected Behavior
Handle the case for $args-count == 3, so get-theme-color does not throw errors when it receives 4 arguments.
Actual Behavior
Function get-theme-color returns a following error:
Function finished without @return.
Environment
- Angular: 18.2.6
- CDK/Material: 18.2.6
- Browser(s): all
- Operating System (e.g. Windows, macOS, Ubuntu): all
Metadata
Metadata
Assignees
Labels
P3An issue that is relevant to core functions, but does not impede progress. Important, but not urgentAn issue that is relevant to core functions, but does not impede progress. Important, but not urgentarea: material/core