Include serde::Deserialize for AttrValue

[bbonenfant]

Request

Include the serde feature from the implicit-clone crate.

Details

I have a use-case where I want a parent component to use the response from the backend to create some child components and in this case I'd like to directly pass parts of the deserialized response within the child component properties. The documentation suggests that you should use AttrValue in place of String where possible, and so it follows that I'd want to deserialize the string values of the response directly into AttrValue, but it's not at clear how one would do that. After doing some investigation I found that the implicit-clone crate, where AttrValue is defined, does implement serde::Deserialize trait but only exposes it when installed using the serde feature.

Since serde is already a dependency for yew, it would be nice to include the ability deserialize into AttrValue by default. If you are open to this change I'm happy to offer to submit a PR, as well as try to update the tutorial example to deserialize directly into AttrValue

[ranile]

What prevents you from adding a dependency on implcit-clone and enabling the serde feature?

[bbonenfant]

Nothing is preventing that. In fact I'm currently doing so to accomplish my goal. I'm just suggesting that including implicit-clone with the serde feature enabled would be helpful to users and there doesn't appear to be any tradeoffs since serde is already a dependency of yew. Alternatively, this information could be included in the documentation if there is a downside to this change that I'm not seeing.

[kirillsemyonkin]

I agree that since Yew has a serde dep unconditionally (if not used for anything internal, consider hiding it behind a feature?), ImplicitClone should have the feature enabled as well.