Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move Overlays logic to an extension #29

Closed
zuck opened this issue Dec 28, 2013 · 2 comments
Closed

Move Overlays logic to an extension #29

zuck opened this issue Dec 28, 2013 · 2 comments
Labels
change

Comments

@zuck
Copy link
Contributor

zuck commented Dec 28, 2013

I propose to move all the Overlays logic to a separated extension.

Why?

Well, basically because:

  1. In real use cases, I think, a stand-alone GUI solution is used instead of basic Overlays.
  2. Overlays are just a restricted use case of, more general, geometry primitives drawing (see: Add extension to render debug primitives #24)
  3. Split the core from extension concepts is usually a winning idea.

Discussion on this thread:

http://www.horde3d.org/forums/viewtopic.php?f=8&t=2162
@zuck zuck added this to the 1.1.0 milestone Feb 17, 2014
@zuck zuck added change and removed enhancement labels Feb 17, 2014
@zuck zuck modified the milestones: 1.0.0 (final), 1.1.0 Feb 17, 2014
@zuck zuck removed this from the 1.1.0 milestone May 24, 2014
@algts
Copy link
Collaborator

algts commented Nov 24, 2017

Done in develop branch.

@zuck
Copy link
Contributor Author

zuck commented Nov 24, 2017

Fix with 4d4263d.

@zuck zuck closed this as completed Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zuck @algts