Component Callback documentation suggests non-existent conversion of closure to Callback

[finnbear]

This is about:

• A typo
• Innaccurate/misleading documentation (e.g. technically incorrect advice)
• Undocumented code
• Outdated documentation
• Other

Problem

Documentation:

Closures or Functions provided as props inside the html! macro are automatically converted to Callbacks.
(https://yew.rs/docs/concepts/components/callbacks)

Reality: <MyComponent callback={|_| ...}>, where Props { callback: Callback<()> } leads to IntoPropValue<yew::Callback<()>> is not implemented for [closure@src/ui.rs:15:35: 15:75]. Manually wrapping closure in Callback::from is a workaround.

Details about the solution you'd like (Optional)

Ideally, passing closures to props of type Callback<_> would be supported, for components, not just elements.

Additional context (Optional)

https://discord.com/channels/701068342760570933/703449306497024049/957453333403152385

In particular, WorldSEnder wrote

the page was probably moved at some point by mistake, since all the examples refer to elements, but it's under "components"

Questionaire (Optional)

• I'd like to write this documentation
• I'd like to write this documentation but I'm not sure what's needed
• I don't have time to add this right now, but maybe later
• I could delete the incorrect phrase from the documentation, but if the solution I like is adopted, I'm not prepared to implement it without guidance

[futursolo]

I would like to see this to be supported.

To add this, you need to add an IntoPropValue implementation.

Something like the following:

impl<I, O, F> IntoPropValue<Callback<I, O>> for F
where
    F: 'static + Fn(I) -> O,
{
    fn into_prop_value(self) -> Callback<I, O> {
        Callback::from(self)
    }
}

Also, to improve API ergonomics, you may also want to implement IntoPropValue for the following conversions:

F -> Option<Callback<I, O>>
Option<F> -> Option<Callback<I, O>>

[finnbear]

I've shamelessly copied your hypothetical implementation, and adaptations of it for the other two cases, into a pull request to hopefully expedite this 😂