Sollumz / Sollumz

Grand Theft Auto V modding suite for Blender. This add-on allows the creation of modded game assets: 3D models, maps, interiors, animations, etc.
https://docs.sollumz.org
MIT License
413 stars 155 forks source link

Add a confirmation dialog when exporting without uv map #952

Open krzysiula3000 opened 3 weeks ago

krzysiula3000 commented 3 weeks ago

or a notification, also when the uv map is empty / not named properly with the index

krzysiula3000 commented 3 weeks ago

same for non power of 2 textures, and other stuff like that where 9999 people ask for help with daily,

this would make place for people actually in need of help or finding bugs

alexguirre commented 3 weeks ago

Now there are warnings in the UV maps and color attributes panels, but I guess a warning on export would be good too. image

krzysiula3000 commented 3 weeks ago

Yea i noticed that and it's a very good change, however there is still so much stuff like that, giving exception errors and confusing people instead of actually being helpful describing what could've went wrong. For example, selecting an action with no keyframes in the ycd gives an exception error, instead of checking for that and giving an appropriate message like "no keyframes in the action" which would be much more helpful, and if the person had at least a braincell they would likely figure it out, eliminating the need to ask in the discord and wait

alexguirre commented 3 weeks ago

Yes, it should never show exception errors without a clear message. Just open a new issue when encountering those cases.

alexguirre commented 2 weeks ago

Ok, haven't added the UV maps/color attributes check on export yet, but made it so the Info Log window opens automatically if an export generated any warnings or errors (a59af379490ee475fa5a0020e0c07f449ccb220e). Should hopefully make it easier for the user to notice that there is something wrong. image