Skip to content
This repository has been archived by the owner on Oct 1, 2021. It is now read-only.

How to ship virtual-list-element.js dependencies? #27

Closed
valdrinkoshi opened this issue Apr 21, 2018 · 7 comments
Closed

How to ship virtual-list-element.js dependencies? #27

valdrinkoshi opened this issue Apr 21, 2018 · 7 comments

Comments

@valdrinkoshi
Copy link
Collaborator

In particular ./layouts/*.js, ./virtual-repeater.js and ./virtual-list.js

@kenchris
Copy link
Contributor

I guess the future work on Web Packaging will solve this? @jyasskin

@domenic
Copy link
Collaborator

domenic commented Apr 30, 2018

I don't think that's quite related. This is more of a question of how or whether we include the lower-level building blocks (detailed in DESIGN.md).

@jyasskin
Copy link
Member

What do you mean by "ship"? If you just stick the files on a server, the natural module loading process will find them, but it'll take more round-trips than optimal. If you bundle just the virtual-list-element.js dependencies, they'll load in one round-trip. If you let the application author bundle virtual-list-element.js and its dependencies with the application itself, they might load in 0 extra round-trips.

@domenic
Copy link
Collaborator

domenic commented Apr 30, 2018

I think the question is much more fundamental than that; sorry about the ambiguous OP.

Right now the plan is to only include <virtual-list> in the layered API. But, maybe we should include the various building blocks---currently named VirtualRepeater, Layout, and RepeatsAndScrolls, and currently located in the files listed in the OP.

The OP's question is: should we include them? If so, what exact APIs are good to expose?

@jyasskin
Copy link
Member

Yep, Web Packaging says nothing about the public API you should expose. :)

@kenchris
Copy link
Contributor

Yeah, I guess the issue title wasn't all that clear :-)

@domenic
Copy link
Collaborator

domenic commented Feb 8, 2019

Given the API shakeups, let's close this for now, and revisit the mechanics of exposing the public API when we have a better idea what the public API actually is :)

@domenic domenic closed this as completed Feb 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants