Skip to content

Proposal: Make MdDialogConfig generic and use type for data property #4398

Closed
@epelc

Description

@epelc

Bug, feature request, or proposal:

I propose making MdDialogConfig a generic and use the type for the data property to add better type safety.

See https://github.com/angular/material2/blob/master/src/lib/dialog/dialog-config.ts#L49

Instead of using any you could do

class MdDialogConfig<T> {
  ...
  data?: T;
}

This would give better type safety on the data property.

What is the current behavior?

Data is type any

Which versions of Angular, Material, OS, browsers are affected?

2.0.0-beta.3 cesium-cephalopod (2017-04-07)

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions