Up to iced 0.4.2 everything was fine, but from v0.5.0 on I have a conflict I'm not able to solve. It is related to a dependency in common between plotters, iced and plotters-iced. I see that you updated the crate and the instruction to the new versions, didn't you face this problem? I am running on MacOS.

error: failed to select a version for `servo-fontconfig-sys`.
    ... required by package `servo-fontconfig v0.5.1`
    ... which satisfies dependency `servo-fontconfig = "^0.5.1"` of package `crossfont v0.5.0`
    ... which satisfies dependency `crossfont = "^0.5.0"` of package `sctk-adwaita v0.4.1`
    ... which satisfies dependency `sctk-adwaita = "^0.4.1"` of package `winit v0.27.0`
    ... which satisfies dependency `winit = "^0.27"` of package `iced_winit v0.6.0`
    ... which satisfies dependency `iced_winit = "^0.6"` of package `iced v0.6.0`
    ... which satisfies dependency `iced = "^0.6.0"` of package `sniffnet v1.0.1 (/Users/giulianobellini/Desktop/GitHub/sniffnet)`
versions that meet the requirements `^5.1.0` are: 5.1.0

the package `servo-fontconfig-sys` links to the native library `fontconfig`, but it conflicts with a previous package which links to `fontconfig` as well:
package `yeslogic-fontconfig-sys v3.0.0`
    ... which satisfies dependency `yeslogic-fontconfig-sys = "^3.0.0"` of package `font-kit v0.11.0`
    ... which satisfies dependency `font-kit = "^0.11.0"` of package `plotters v0.3.4`
    ... which satisfies dependency `plotters = "^0.3"` of package `plotters-iced v0.5.0`
    ... which satisfies dependency `plotters-iced = "^0.5.0"` of package `sniffnet v1.0.1 (/Users/giulianobellini/Desktop/GitHub/sniffnet)`
Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the links ='servo-fontconfig-sys' value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

failed to select a version for `servo-fontconfig-sys` which could resolve this conflict

When creating a simple gui with a text box and a graph, when setting widths/lengths for both, the program fails to compile saying

expected enum iced_core::length::Length, found enum Length.

It suggests that multiple versions of iced_core are being used. I'm using iced version 0.4, which uses iced_native 0.5. I looked, and it seems like this repository is using iced_native 0.4. I think this is preventing people from using newer versions of iced in conjunction with plotters-iced. Is there any possibility this repository will be updated? Or is there another issue here?

iced 0.5 was recently released. This causes (if you try to use iced 0.5 in your project) type mismatches with, for example, iced_core::length::Length (mismatch between iced_core 0.5 and 0.6), thus making it impossible to use iced 0.5 with this library.

The pure way of doing things seems to be the default now: https://github.com/iced-rs/iced/pull/1393

Note that this change hasn't been released yet, but it's coming so just a heads-up. 🙂

There are no examples given for the pure version of this widget and it seems that the pure version also requires mutable access to state object from fn view.

Is the pure version incomplete? An example would be much appreciated.

Commit af59edc1b673a86656aadd95ae3980503f4e7323 addresses the following lints:

• upper_case_acronyms
• while_let_on_iterator
• branches_sharing_code

Commit dabfd8db95b2f54dd708377e465eaca0f5f45dff and 3983f42c070099a2d08e6e13bb41dbc2ee149442 fix other small pedantic things I noticed while making other changes.

The meat of the change is in commit b95f26823e40b39dec156c1c74f6231fcdb615e2. This commit does not restrict the chart passed to the ChartWidget to a &mut reference to something that implements Chart. This commit instead allows the ChartWidget to be generic over any type that implements Chart and also adds a blanket implementation over all &mut T where T: Chart. These two changes in conjunction allow for the ChartWidget to take either a mutable reference to or ownership of a type that implements Chart.

The previous implementation of ChartWidget to always take a &mut Chart makes sense in most use cases since the user is most likely to create a struct to hold persistent data about the chart and pass a mutable reference to that to the chart widget in the view method on the iced::Application. My use-case however required that I pass extra data to all of the charts in the update function. To accomplish this, I needed to return a new structure with the new data that will implement Chart, but this was not possible since the ChartWidget implementation required a reference to this temporary structure that could not persist.

I have not thoroughly tested the change in implementation to ensure it will not break any existing code though my idea was to ensure this would be 100% backwards compatible with the previous implementation. I have tested the implementation on both of the native examples.

I did not implement this change on the web implementation due to my lack of familiarity with its implementation and the implementation seeming reliant on the chart passed as a reference as seen here:

https://github.com/Joylei/plotters-iced/blob/master/src/web/widget.rs#L97

I was unable to compile using a Theme that I defined myself instead of using iced_style::Theme which was added in iced 0.5.

error[E0277]: the trait bound `iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>: plotters_iced::renderer::Renderer` is not satisfied
   --> src/panel/pqn.rs:698:9
    |
698 | /         plotters_iced::ChartWidget::new(self)
699 | |             .width(iced::Length::Units(40))
700 | |             .height(iced::Length::Fill)
    | |_______________________________________^ the trait `plotters_iced::renderer::Renderer` is not implemented for `iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>`
701 |               .into()
    |                ---- required by a bound introduced by this call
    |
    = help: the trait `plotters_iced::renderer::Renderer` is implemented for `iced_graphics::renderer::Renderer<B, iced::Theme>`
    = note: required for `iced_native::Element<'_, Message, iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>>` to implement `From<ChartWidget<'_, Message, iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>, &LegendWidget>>`
    = note: required for `ChartWidget<'_, Message, iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>, &LegendWidget>` to implement `std::convert::Into<iced_native::Element<'_, Message, iced_graphics::renderer::Renderer<iced_glow::backend::Backend, MyTheme>>>`

The reason is that the plotters_iced::renderer::Renderer trait is limited to implement for iced_graphics::Renderer<B, iced_style::Theme>.

In the patch, the concrete type iced_style::Theme was changed to a type parameter.

By enabling and disabling the pure and native features for this crate, the ChartWidget can be either used in impure applications or pure applications.

Thanks for this super useful library! It's made it extremely easy for me to make an interactive visualization application.

I was hoping to incorporate user interaction with the diagram into my application, but it seemed that the plotters-iced ChartWidget isn't doing anything with mouse events. This PR adds the ability for users to listen for mouse events that take place in their ChartWidgets.

I've also added an example of using this feature to make a simple grid that can be sketched on (see picture below).