linebender/druid

Decouple piet and druid-shell

Open

#1924 aperta il 16 ago 2021

Vedi su GitHub
 (8 commenti) (1 reazione) (0 assegnatari)Rust (567 fork)batch import
architecturehelp wanted

Metriche repository

Star
 (9091 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

It is not currently possible to use druid-shell without using piet, and I think this is an unnecessary coupling, and it makes druid-shell strictly less useful.

I would like to move all of the piet-specific code from druid-shell into druid. In practice this means:

  • Any shared PietText objects should live somewhere in druid (unclear where, exactly, druid doesn't really have an application-global object? maybe the AppHandler?)
  • instead of passing a RenderContext to the paint method, druid-shell should pass a window handle
  • druid should construct the RenderContext itself, from the window handle

This does violate the principal that platform-specific code should generally be in druid-shell instead of druid. Is there a viable alternative? I'm not sure if there's a good one, although we could perhaps have a from_raw_handle(handle: &RawWindowHandle) -> RenderContext method in piet-common? I'm not sure if this really makes a ton of sense, but it's an option?

If someone is interested in taking this on, I am happy to answer questions.

Guida contributor

Direzione di ricerca
Indagare dove gli oggetti PietText sono attualmente utilizzati in druid shell e come sono condivisi. Esaminare la modifica del metodo paint per passare un handle di finestra invece di un RenderContext, e quindi costruire il RenderContext in druid da quell'handle. Considerare l'alternativa di aggiungere un metodo from raw handle a piet common. Rivedere i commenti dell'issue per eventuali considerazioni di progettazione aggiuntive.
Tech stack
rust
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30