GraphiteEditor / GraphiteEditor/Graphite

Make the Rasterize node resolution-aware

オープン
#3,391 コメント 0 件 リアクション 0 件 担当者 1 名 @TrueDoctor が担当を希望しています GitHub で見る
Graphics
主要言語
Rust
スター
27.2k
フォーク
1.3k
平均マージ
20時間 5分
マージ済み PR(30日)
57

説明

The current Rasterize node is a major footgun because its bounds have to be set manually, which multiple users have complained about feeling broken because they didn't realize a width, height, and X/Y offset had to be chosen to make use of the node.

We want to instead use the minimum of the content bounds and the viewport bounds (the context's footprint), and with a resolution assigned by the viewport resolution from the context's footprint. This lets us eliminate the bounds parameter of the Rasterize node so that node works without needing any parameters.

This makes the "Blend" node no longer necessary, because that is simply equivalent to a stack of two raster layers that feed into a Rasterize node. So that node can also be removed.

To allow the user to choose rendering bounds, we also need a separate Crop node. A user would apply a Rasterize node followed by a Crop node to achieve the same outcome. Perhaps also a Resample node to change the rendering resolution for upsampling/downsampling.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。