code and ui cleanups - Githubissues

stuffmatic / blam

🚨 No longer maintained. Check out fSpy instead 🚨 A camera and projector calibration toolkit for Blender that also does automatic 3D reconstruction of geometry with rectangular faces.

https://fspy.io/

GNU General Public License v3.0

644 stars 95 forks source link

code and ui cleanups #5

Open dfelinto opened 10 years ago

dfelinto commented 10 years ago

1) all UI parameters need to start with a captalized letter 2) created scene.blam to keep scene namespace clean 3) stripped whitespaces out 4) add two lines separation between classes 5) replaced some camelcase variables by underscore separated 6) cleaned UI code (no need for rows

dfelinto commented 10 years ago

I'm trying to use BLAM for a project, and for that I will need to do some changes.

I don't know if my changes will be 'mergeable' since it may be too hacky, but either way I wanted to cleanup the script a bit to understand it better.

So I thought I may as well pull request the cleaning up, in case you find it useful. cheers

aceads0529 commented 10 years ago

I'm suggesting some general UI cleanups

Replacing long button names with short ones (tool-tips explain the functions adequately)
Reduce box usage (they add clutter)
Consistency in capitalization
Restriction to 2.70+ tabs ("Tools" in 3D View, "Solve" in Movie Clip Editor)

Thanks!

durnievihor commented 6 years ago

not even installing