haunted icon indicating copy to clipboard operation
haunted copied to clipboard

Context and virtual components

Open orlov-vo opened this issue 6 years ago • 7 comments

Now we have a React-like context (see issue #17). But we can't use it in an application built with virtual components only (without web components).

I think we should provide some directives for lit-html after creating context. However, we need to discuss and choose the comfortable api for it...

An abstract example of usage:

const ThemeContext = createContext('dark');

const { provide, consume } = ThemeContext;

const MyConsumer = virtual(() => {
  const context = useContext(ThemeContext);

  return context;
});

const App = virtual(() => {
  const [theme, setTheme] = useState('light');

  return html`
      <select value=${theme} @change=${(e) => setTheme(e.target.value)}>
        <option value="dark">Dark</option>
        <option value="light">Light</option>
      </select>
      
      ${provide(theme, html`
        ${MyConsumer()}
        ${provide(theme === 'dark' ? 'light' : 'dark', html`
          ${consume((value) => html`<h1>${value}</h1>`)}
        `)}
      `)}
  `;
});

render(App(), document.body);

orlov-vo avatar Dec 06 '18 15:12 orlov-vo

@askbeka Can you remember what the problem was with context in virtual components?

matthewp avatar Dec 06 '18 18:12 matthewp

Yes, problem is with unsubscribing from context changes, since virtual components do not have a knowledge when they are removed, neither do parts. One of the way could be to put text or comment element and check if it still present in DOM in interval.

I am workings through couple of ideas, for DOM traversal, to find virtual provider, from virtual consumer, which are abstractions of collection of nodes they cover in templates. But events are no longer beneficial in that case, it is better to just traverse manually, and it is faster too.

askbeka avatar Dec 07 '18 03:12 askbeka

Can we use a MutationObserver to know when it's removed?

matthewp avatar Dec 08 '18 02:12 matthewp

@matthewp Not really. since directive do not have their own parts, parts can be reused across directives. consider:

html`${predicate ? ComponentOne() : ComponentTwo()}`

they will use the same part and the same marker nodes, which means that when Component is no longer used markers will not be removed.

Exmaple above will not work with virtual components, I have addressed it in this issue

Only way to do this is to associate part with directive, so that when different directive is used with the same part, previous directive gets revoked.

Same should happend when previous part gets removed, I will file an issue, future request to lit-html, see if they have some solution in mind

askbeka avatar Dec 14 '18 12:12 askbeka

Any news about it? 😁

djalmajr avatar Jun 24 '20 15:06 djalmajr

Polymer/lit-html#283

djalmajr avatar Jul 19 '20 23:07 djalmajr

Polymer/lit-html#283

Issue is closed 🚀

ianldgs avatar Dec 28 '20 01:12 ianldgs