JoernT
commented
1 year ago A working solution has been pushed to dev branch. A new demo illustrates usage with different tables and lists.
However there's a limitation: repeats via attributes cannot be nested at this point in time. As the implementation gets significantly more complex when nesting tables i've decided to wait for user feedback and defer this step until a use case comes up. This limitations does NOT apply to usual repeats - those can be nested within a table created via attributes.
A second reason to defer implementation of nested tables is their rather bad performance in browsers - it takes significantly more time for a browser to paint tables than e.g. nested div structures. This mostly occurs when tables have no fixed dimentions and need several paint runs to get calculated dynamically expecially when they contain other dynamically layed out elements like select elements.
implementation notes
Instead of merging the functionality now a new fx-repeat-attributes is used. This share significant portions of the implemenation with fx-repeat and might be merged at a later stage again. This road was mainly taken to not interfere with the current fx-repeat element implementation.
nverwer
Discussed in https://github.com/Jinntec/Fore/discussions/38
``` Using fx-repeat as is would force: ```
- {.}
``` Not at all want you want - breaks content-model of `` and may even break styling or require workarounds. So in consequence what we'll need is: ```
::shadowDOM ...
- 1
- 2
- 3
```
`fx-repeat` would become an invisible wrapper around the actual repeated element and:
* move `data-ref` to `fx-repeat/@ref`
* move `` to shadowDOM of repeat
* unroll items
Same principle should work for tables or other similar structures.
- {.}
``` ## Possible implementation At initial refresh the fx-form: * querySelectorAll('[data-ref]') * create fx-repeat for each of them like so: ```