-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
voidbert
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll put ImGUI in the lib directory first, and then leave you to these changes
Ubuntu's packaging solution is quite weird
voidbert
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I just added ImGUI and I was wondering if it would be better or not to put the UI class in the window namespace. I'm not seeing the ui namespace getting more classes in the future, and a namespace with a single class defeats the purpose of namespaces. What's your opinion on this?
Hi! I agree with you, it makes sense. I'll go ahead and make that change now. |
Uh oh!
There was an error while loading. Please reload this page.