-
Notifications
You must be signed in to change notification settings - Fork 18
Can the list be generated? #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Sorry I totally thought I had replied to this. I've just released v2.0.0 and this feature is included (somewhat). As of 2.0.0 you can pass a synchronous function instead of a static array of colors. I'm still working on 2.0.1 which will support async, will release it as soon as I work out some kinks. Thanks for the great idea! |
@KimPaow any update on adding async support to this? I'm hoping to pull the list of colors from an array in a Settings document, where editors can choose their own pre-defined colors with color-input :) here's what i'm trying to do in my schema: {
title: 'Color List',
description: 'Pick a color',
name: 'colors',
type: 'colorlist', // required
options: {
list: async () => {
const response = await client.fetch(`*[_type == 'settings' && _id == '_settings'][0].colors`);
const colors = response.reduce(
(acc: any, color: any) => {
acc.push({
value: color.hex,
title: color.hex,
});
return acc;
}, []);
console.log(colors);
return colors
}
},
}, |
I just hacked together a quick fix for this. @devinhalladay could you try and install |
Wow, awesome @KimPaow — thanks for the quick work! Sadly I'm getting an uncaught error:
|
I was able to get my schema and groq query (as shown in my previous comment) working via a couple simple changes I've pushed as a branch here. I simply made EDIT: I downloaded the repo as of your last commit and pasted the files into my Aside from the error I'm getting from the hosted package, |
Ah okay got it, seems like I need to set up some polyfills for babel (which is why the source files are fine and the build breaks). Nice that you got a temp fix though! |
@KimPaow any idea when you may be able to action on this? If you're short on time, and could let me know what polyfills need to be added, I'd be down to try to put up a PR! Really appreciate your work, this project is awesome. |
@devinhalladay sorry for the delay. I would like to fix this asap but have too much on my plate right now. I could probably fix it in another two weeks or so. If you would like to put together a PR that would be very much appreciated! :) |
I would like to pick a color from the color palette of an image asset.
So, I was thinking, could the color list be generated?
For example by passing a function instead of an array to the
list
option? I guess it should be an async function so that you could look up the asset reference and generate a color list.Does it make sense?
Nice plugin @KimPaow !
The text was updated successfully, but these errors were encountered: