Change shader traits to handle ID enums instead of integers

[thatcomputerguy0101]

This is the breaking change half of #484, where all shader traits pass IDs as their open_enum types (technically structs) instead of as integers. open_enum allows custom IDs to be added by the consuming program, as seen in the firework, logo, and Mandelbrot examples.

I noticed that while changing the original reserved ID comments on Effect that it recommended using the IDs above the internal range instead of below the internal range. I suspect that is due to how materials and effects share the same ID space. If that system is desired to be kept, the documentation of EffectMaterialId will need updated accordingly. Otherwise, it might be good to compress the ID usage of the built-in effects to all be above 0x8000, leaving the lower half of the ID range for public use similar to the other two ID types.

[asny]

Makes sense 👍 Again, thanks for your contribution 🙏

I noticed that while changing the original reserved ID comments on Effect that it recommended using the IDs above the internal range instead of below the internal range.

Yeah, I think I combined the IDs in a different way at some point.