Skip to content

Conversation

@marc2332
Copy link
Member

@marc2332 marc2332 commented Mar 10, 2024

Prepare support for Dioxus 0.5

Closes #26
Closes #28
Closes #14

@marc2332 marc2332 added the enhancement New feature or request label Mar 10, 2024
@marc2332
Copy link
Member Author

marc2332 commented Mar 10, 2024

Looks like material design icons are failed to be written, looking into it Fixed

@iynaix
Copy link

iynaix commented Mar 26, 2024

Trying to use it with the following in Cargo.toml:

dioxus-free-icons = { git = "https://github.com/dioxus-community/dioxus-free-icons", branch = "feat/dioxus-0.5", features = [
    "material-design-icons-editor",
] }

I'm getting hundreds of errors such as:

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1816:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1816 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1837:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1837 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1858:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1858 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

Some errors have detailed explanations: E0119, E0428.
For more information about an error, try `rustc --explain E0119`.
error: could not compile `dioxus-free-icons` (lib) due to 595 previous errors
warning: build failed, waiting for other jobs to finish...

Am I missing something obvious here like having to run a codegen step?

@marc2332
Copy link
Member Author

Trying to use it with the following in Cargo.toml:

dioxus-free-icons = { git = "https://github.com/dioxus-community/dioxus-free-icons", branch = "feat/dioxus-0.5", features = [
    "material-design-icons-editor",
] }

I'm getting hundreds of errors such as:

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1816:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1816 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1837:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1837 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1858:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1858 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

Some errors have detailed explanations: E0119, E0428.
For more information about an error, try `rustc --explain E0119`.
error: could not compile `dioxus-free-icons` (lib) due to 595 previous errors
warning: build failed, waiting for other jobs to finish...

Am I missing something obvious here like having to run a codegen step?

No, it looks like the codegen generated is wrong, will check asap! Thanks for the report!

@marc2332
Copy link
Member Author

Trying to use it with the following in Cargo.toml:

dioxus-free-icons = { git = "https://github.com/dioxus-community/dioxus-free-icons", branch = "feat/dioxus-0.5", features = [
    "material-design-icons-editor",
] }

I'm getting hundreds of errors such as:

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1816:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1816 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1837:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1837 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

error[E0119]: conflicting implementations of trait `IconShape` for type `md_editor_icons::MdMaterialDesignIcons`
    --> /persist/cache/home/iynaix/.local/share/.cargo/git/checkouts/dioxus-free-icons-d246d2ebe6d920da/58f601c/packages/lib/src/icons/md_editor_icons.rs:1858:1
     |
6    | impl IconShape for MdMaterialDesignIcons {
     | ---------------------------------------- first implementation here
...
1858 | impl IconShape for MdMaterialDesignIcons {
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ conflicting implementation for `md_editor_icons::MdMaterialDesignIcons`

Some errors have detailed explanations: E0119, E0428.
For more information about an error, try `rustc --explain E0119`.
error: could not compile `dioxus-free-icons` (lib) due to 595 previous errors
warning: build failed, waiting for other jobs to finish...

Am I missing something obvious here like having to run a codegen step?

Try now! I think it should work

@iynaix
Copy link

iynaix commented Mar 26, 2024

It seems to work now (no more build errors), but I am running into #26. There's an empty path over the actual icon so it always shows up as a filled box. :(

@marc2332
Copy link
Member Author

It seems to work now (no more build errors), but I am running into #26. There's an empty path over the actual icon so it always shows up as a filled box. :(

Will try to fix asap!

@marc2332
Copy link
Member Author

marc2332 commented Mar 26, 2024

It seems to work now (no more build errors), but I am running into #26. There's an empty path over the actual icon so it always shows up as a filled box. :(

Try now! I think I got it working 😄 I also updated it to alpha 2

@iynaix
Copy link

iynaix commented Mar 26, 2024

It's working great now! ❤️

@marc2332
Copy link
Member Author

It's working great now! ❤️

Awesome!

@marc2332 marc2332 marked this pull request as ready for review March 28, 2024 17:14
[dependencies]
dioxus = { version = "0.4.0" }
dioxus-web = { version = "0.4.0" }
dioxus = { version = "0.5.0-alpha.0", features = ["web"] }
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.5.0

@marc2332 marc2332 merged commit 14493cc into main Mar 30, 2024
@marc2332 marc2332 deleted the feat/dioxus-0.5 branch March 30, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inner stroke style on ionicons Material design just draws box instead of icon shape Unnecessary title tags created in HTML

4 participants