Set default density to 1.25 #1146
Conversation
Lower the default density to avoid ugly visual artifacts. On Pico and Quest 2, pages and images with very thin lines cause distortions and Moiré patterns when the density is too high. Lowering the density eliminates these effects.
felipeerias
requested review from
elima,
svillar,
javifernandez and
HollowMan6
|
The visual issues can be appreciated with images like this one:
https://as1.ftcdn.net/v2/jpg/02/07/01/90/1000_F_207019089_pWCeZirf2dUaSXWmf1sbNS5hezjYX9gC.jpg This is how this image is displayed on the Quest 2 with density 1.5. Detail showing the Moiré pattern:
This is how the image looks with density 1.25.
Detail showing that the visual effect is gone:
It is hard to describe in a comment, but this situation looks much worse in reality because each eye sees a different effect. |
There was a problem hiding this comment.
Aliasing in graphics is something we do need to consider, so that's why I mentioned here that it's good to have a default value for each kind of device to match the device resolution instead of specifying something default for all. #1086 (comment)
But anyway I can give a pass for this PR, maybe we can do that later. Unfortunately, I only have Meta Quest 2 so I can't really help with this issue.
|
Thanks @HollowMan6. Yes, we need a complete solution with different values for each device and which also makes it easier for users to change those values. For the moment, this is a safe default for the 1.5.2 release. |
Lower the default density to avoid ugly visual artifacts.
On Pico and Quest 2, pages and images with very thin lines cause distortions and Moiré patterns when the density is too high. Lowering the density eliminates these effects.