Jump to content

Elimination of TargetCamera into Camera


Recommended Posts

Actually I thought about it twice and I do not see a real value to remove it. It helps having smaller classes (think about mesh class). Camera is fat enough :) And who knows, perhaps we may find a kind of camera without a target (Ok I doubt about it :))


Is there a problem on your side this merge could fix?

Link to comment
Share on other sites

None.  My problem / bias against TargetCamera was when ArcRotateCamera did not inherit from it (against ArcRotateCamera really).  Made refactoring 3D rigging directly into Camera & getting rid of most of those VR cameras more difficult.  Brought it up, just so leaving it alone was an active decision.  As repository grows, things can get more and more layered.  Not always a good thing.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    • No registered users viewing this page.
  • Create New...