Feature idea: modifier/node interpolator

[ansh7432]

Add nodes string interpolator for mixing text and dynamic content

Summary

This PR implements the nodes string interpolator feature requested in #401. It enables mixing static text with dynamic content (such as signals and modifiers) in a clean, intuitive way.

Implementation Details

The implementation adds a StringContext extension method called nodes that allows for string interpolation with various types:

Signal[F, String] and Signal[F, ?] for reactive values
Modifier[F, El, ?] for existing modifiers
Product types (tuples and case classes)
Effect types like F[String]
Standard values (using toString)
Each interpolated string becomes a series of text nodes and dynamic content, preserving the reactivity of signals while providing a more ergonomic API.

Example Usage

##Benefits
Improved readability: Code is more concise and follows natural expression patterns
Maintained reactivity: Only the signal parts update when their values change
Type flexibility: Works with various types beyond just Signals
Intuitive syntax: Familiar string interpolation pattern from standard Scala

[ansh7432]

Hii @kubukoz @armanbilge please review the PR i had made the changes

[armanbilge]

Thanks for the PR! To test it works, can you update the TodoMvc demo app to use this new feature?

[ansh7432]

Thanks for the PR! To test it works, can you update the TodoMvc demo app to use this new feature?

Yeah sure making these changes.. :)

[ansh7432]

Hii @armanbilge @kubukoz getting some error while implementing the test for nodeinterpolator please go through the code and review it

[kubukoz]

Can you push the code with the broken test? The last CI run is green

[ansh7432]

Hi @kubukoz try to add test in todomvc file adding nodesinterpolator please check and review when you have time I've made these changes:

Added the import: import calico.html.nodes
Updated the label in TodoItem to use the interpolator:
label(nodes"${todo.map(.map(.text).getOrElse(""))}")

When compiling, I'm seeing errors related to the nodes string interpolator. Initially, I tried using it with type parameters like nodes[IO]"...", but that caused syntax errors. When removing the type parameters, it still fails to compile.

[armanbilge]

@ansh7432 I think your implementation is quite complex. I recommend to start from the initial proposal in #401 (comment) and support additional features one-by-one, instead of all at the same time. That will make it easier to debug.

[ansh7432]

Hii @armanbilge @kubukoz implemented nodesinterpolator test in todoMvc and its working fine please check if needs more changes :)

[armanbilge]

and its working fine

Hmm, are you sure? I just gave it a try, and the TodoMVC app seems to be broken, I am unable to add any todos 😕

For reference, here is the working example: https://armanbilge.github.io/calico/todomvc/

[ansh7432]

hii @armanbilge I apologize, please check it once more its working in my local machine now..

[ansh7432]

Hi @armanbilge @kubukoz gentle reminder to review this 😊

[ansh7432]

Thanks for the review fixing all the things and again raising pr soon 😊

[armanbilge]

Nice work, this is much improved! I think like Jakub suggested in #441 (comment), the next step is move more of the interpolation into compile-time.

In general, I think the runtime class checks and casts are going to cause problems down the line - the interpolator should know the types of its inputs at compile-time.

[ansh7432]

Nice work, this is much improved! I think like Jakub suggested in #441 (comment), the next step is move more of the interpolation into compile-time.

In general, I think the runtime class checks and casts are going to cause problems down the line - the interpolator should know the types of its inputs at compile-time.

Yeah so looking for this now after working on this would this be able to merge ?

[armanbilge]

Now that we've prototyped the idea, let's keep improving it in this PR until it's production-ready 💪

[ansh7432]

Yeah sure will working on this @armanbilge 😃

[ansh7432]

hii @armanbilge I had updated the PR please look into this.. 😊 please suggest more improvements on this 😀

[ansh7432]

Hii @kubukoz @armanbilge please review this :)