Any plans to support compose resources and passing resources directly without creating IconKey?

# jewel

[mohamed rejeb]

Any plans to support compose resources and passing resources directly without creating IconKey?

[seb]

You can use the standard Compose for Desktop APIs for interacting with Compose Resources — IconKey is not going to go away as a way to accessing icons in Jewel as it's a necessary abstraction over IJP's icon loading to make it portable between standalone and bridge modes.

[seb]

In fact as of 0.25.0, Jewel uses Compose Resources under the hood when loading from IconKeys

[mohamed rejeb]

What about having function overloading? Each component that accepts an IconKey can accept a painter, bitmap...

[seb]

You can use the existing Compose for Desktop APIs for that

[seb]

Unless you want to also use the PainterHints API

[seb]

but that is not a scenario we're considering for now

[seb]

since several PainterHints have no meaning/can't work in scenarios where you pass in a painter/imagebitmap/... since they need access to a file path, or the SVG code

[seb]

Icon(painter, contentDescription) would not be any different from using Image(painter, contentDescription) at that point

[seb]

The Icon API is meant to be used for SVG icons, loaded from resources

[seb]

It doesn't really mean much in other scenarios

[seb]

Hope I have explained the reasoning well enough 🙂

[Chris Sinco [G]]

Each component that accepts an IconKey can accept a painter, bitmap...

Are there specific UI components that you are NOT seeing a slot based API that would accommodate using different Icon composables?

[mohamed rejeb]

Thanks for the explanation, totally makes sense for Icon

[mohamed rejeb]

But there are other components such as SelectableIconActionButton that only accepts IconKey

[seb]

ah, yeah, that is a good point. Could you file an issue requesting a slot-based version of the impacted components?

[seb]

I can't promise we'll get to it right away but we should eventually

[mohamed rejeb]

Ok cool, maybe I can even create a PR for that.

[seb]

That'd be even better! :) we'd appreciate the contribution