Re: [csswg-drafts] [css-values] A way to dynamically construct custom-ident and dashed-ident values (#9141)

> @bramus Nit, but I think this should be sufficient?
> 
> ```
> <ident-fn> = ident(<ident-args>+);
> <ident-args> = [<string>  | <ident-fn>]+
> ```

I see you have a double `+` in the syntax … I think the one after `<ident-args>+` can be dropped, no? Or maybe you meant `ident(<ident-args>#)` to allow multiple options _(in which the 1st non-empty one wins)_?

> * `attr()` returns a string, so it's already included in `<string>`
> * Provided the `var()` returns a `<string>`, it should be already allowed?

That depends on what `<attr-type>` was passed into `attr()` in the first case and how the property was registered in the second.

> * What is the use case for `<ident-fn>` as an arg to another `<ident-fn>`?

I had a good reason but can’t see to recall it right now … hmmm.




-- 
GitHub Notification of comment by bramus
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/9141#issuecomment-2433476893 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 23 October 2024 21:14:51 UTC