cursor-icon version 1.0.0

[kchibisov]

--

r @i509VCB @notgull ?

[madsmtm]

Let's add a no-op alloc feature first, something like:

[features]
default = ["std"]
std = ["alloc"]
alloc = []

// For forwards compatibility
#[cfg(feature = "alloc")]
extern crate alloc as _;

Details.

[kchibisov]

But adding a feature is not a breaking change?

Haven't noticed your link.

[kchibisov]

@madsmtm done in b384d99

[kchibisov]

Adding aliases is not justified, see https://www.freedesktop.org/wiki/Specifications/cursor-spec/ . There's a movement to use standard icon names, so we can keep everything the way it is. And most cursor themes have all the variants listed in that spec already.

[notgull]

I went through the API Guidelines Checklist and made PRs for the things I found missing. Aside from that, looks good to me!

[notgull]

Looks good to me!

[i509VCB]

I'll mention this from matrix given that went into legal off topic:

Should we include documentation for CursorIcon::from_str to specify what string values are expected. I'd guess it's the CSS values from quickly searching?

[kchibisov]

Should we include documentation for CursorIcon::from_str to specify what string values are expected. I'd guess it's the CSS values from quickly searching?

Done in 92dd3e6