Skip to content
This repository was archived by the owner on Jan 14, 2020. It is now read-only.

Conversation

@dfelinto
Copy link

@dfelinto dfelinto commented Oct 9, 2013

  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

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
Copy link
Author

dfelinto commented Oct 9, 2013

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
Copy link

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
Copy link

not even installing

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants