Support default properties
- Dominant language
- TypeScript
- Stars
- 623
- Forks
- 78
- PR merge metrics
- No merged PRs in 30d
Description
@agubler commented on [Wed Jul 26 2017](https://github.com/dojo/widget-core/issues/621)
**Enhancement**
It could be useful for a widget author to be able to specify default properties that the consumer properties are mixed over ensuring that the `this.properties` type guard reflects the defaulted properties so that they are not assumed to be possibly `undefined`.
---
@pottedmeat commented on [Thu Aug 17 2017](https://github.com/dojo/widget-core/issues/621#issuecomment-323215412)
This could be helpful for the dgrid editing functionality where there is a set of properties that should be used across all instances - things like length, style, etc. - but each row has its own value.
---
@agubler commented on [Fri Aug 18 2017](https://github.com/dojo/widget-core/issues/621#issuecomment-323217840)
@pottedmeat unfortunately it's very difficult to provide the mechanism and support all the typings effectively at the moment.
---
@pottedmeat commented on [Fri Aug 18 2017](https://github.com/dojo/widget-core/issues/621#issuecomment-323218015)
That's fine, I was just outlining a use case.
---
@agubler commented on [Fri Aug 18 2017](https://github.com/dojo/widget-core/issues/621#issuecomment-323218730)
Of course, I think there are multiple use cases for default properties one of which is dgrid. Just wanted to let you know where I had got with trying to implement the feature... it led me down a rabbit hole of typings that I couldn't get quite right unfortunately.
---
@kitsonk commented on [Tue Aug 29 2017](https://github.com/dojo/widget-core/issues/621#issuecomment-325635326)
This seems to be related: Microsoft/TypeScript#16173
There appears to still be some challenges, though it seems like the problem space is better understood. We should think about it and try to make sure we have sufficient issues on TypeScript to help solve the problem (or support already existing ones).
Contributor guide
Assessment
This issue has not been assessed yet.