Didn't touch rna, also to make sure that the Python API does not break.
Maybe I missed some, but these don't seem to be used anywere internally.
Differential D5034
Remove unused particle settings from UI Authored by Jacques Lucke (JacquesLucke) on Jun 6 2019, 3:50 PM.
Details Didn't touch rna, also to make sure that the Python API does not break. Maybe I missed some, but these don't seem to be used anywere internally.
Diff Detail
Event TimelineComment Actions Did you remove those settings because they did not work ? Except rendering features that were relatives to Blender Internal, those features are useful and used in 2.79. Hair Grid is a huge help when trying to set-up Voxel Grid Cell Size of Hair Dynamics. That is a must-have for people wanting to use Hair Dynamics. Trails are allowing user to easily create a trail of particles following one particle's path. That is really useful. Animations of particles following a path are very common. If Line rendering is no more supported at all, why just suppresing its settings and keeping the Line type in Particle Rendering list ? Please, don't suppress display properties. They are overlays for particles. Velocity of particles are the same kind of useful overlay when working with force fields, collisions. That helps to find correct settings for particles physics. Number is also used to debug. If particles are not at expected places, it helps user to know if he has to deal with old particles emited at start of simulation or with more recent ones. I understand that Blender Internal rendering types are no more supported. But most of things that you have suppressed, here, are more about helping user to create easily particles animations than how to render them. Comment Actions My understanding is that everything that was hidden was doing nothing in 2.80. If this is not the case, it would be good to get a specific example. Adding back support for some of these would be good, but it will not happen for the 2.80 release and so there is no point having them in the UI. Comment Actions
Those settings were not hidden before this commit. They were just not working. Because nobody took time to make them work. That was not obvious during UI freeze period that no work at all would be done to provide overlays for particles. But here, you are just suppressing what you never gave attention to. Most of those settings are just overlays to add to viewport. They are not rendering types. But if people want to use Halo Type just with OpenGL render, why not considering trails support ? It is just a question of providing some visual info to user. It is not related to complex physics or EEVEE/Cycles rendering of particles. Comment Actions I removed the settings from the UI (they still exist internally), because they are not doing anything in Blender 2.80 currently. Nobody says that they are not useful. We just don't have the manpower to make them work for the Blender 2.80 release. It does not make sense to keep them in the UI if they do nothing. Comment Actions In that case, Line Rendering type should be hidden, too. It is not supported by EEVEE/Cycles or viewport. Comment Actions OK. That is not clean. But at least, is it possible to obtain from @Jeroen Bakker (jbakker) to make Size display working ? |