DateTimeDefaultWidget should implement ContainerFactoryPluginInterface

There was once a FormatterFactory too, which IIRC @tim.plunkett removed in the same patch that introduced the @FieldFormatter plugin annotation.
So maybe similarly this could be done in #2035315: Add a dedicated @FieldWidget annotation ?

tagging

is this issue still valid?

Nope, this was done in #2035315: Add a dedicated @FieldWidget annotation

There's @todo left

Shouldn't we actually inject it then instead of just removing the @todo? Also, that @todo just says inject once this issue is in, so this issue was a blocker for that but there's no reason to do re-open this issue?

@berdir so file new issue? suppose there's no reason to inject EM here, ince there's a method

5: 2035317-5.patch queued for re-testing.

Proper title and summary -we need to resolve @todo

Still think this should have been a different issue, I'm sure there are other formatters and widgets that use global services too that are simply missing a corresponding @todo.

I am rerolling the patch. There were conflicts from #2005434: Let 3rd party modules store extra configuration in EntityDisplay.

@Berdir, I will see if I can find more such widgets and create issues.

Committed c679460 and pushed to 8.x. Thanks!