bluecadet / Cinder-BluecadetViews

Scene graph Cinder block with touch management and various tools for UI dev
MIT License
11 stars 3 forks source link

Create Cinder app template; integrate ClangFormat #134

Closed shi-weili closed 5 years ago

benjaminbojko commented 5 years ago

Great, Weili, can you remove the addTouchParams line then?

On Thu, Nov 8, 2018, 1:50 PM adielfernandez <notifications@github.com wrote:

@adielfernandez commented on this pull request.

In templates/Bluecadet App/src/_TBOX_PREFIX_App.cpp https://github.com/bluecadet/Cinder-BluecadetViews/pull/134#discussion_r232017837 :

  • // SettingsManager::setInstance(myApp::MyAppSettingsManager::getInstance());
  • // Initialize the settings manager with the cinder app settings and the settings json
  • SettingsManager::getInstance()->setup(
  • settings,
  • ci::app::getAssetPath("settings.json"),
  • [](SettingsManager *manager) {
  • // Optional: Override json defaults at runtime
  • manager->mFullscreen = false;
  • manager->mWindowSize = ivec2(1280, 720);
  • }); +}
  • +void _TBOX_PREFIX_App::setup() {

  • BaseApp::setup();
  • BaseApp::addTouchSimulatorParams();

@benjaminbojko https://github.com/benjaminbojko Sure thing! Will pull the changes from WD and fold them into Views when I get back to NYC.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bluecadet/Cinder-BluecadetViews/pull/134#discussion_r232017837, or mute the thread https://github.com/notifications/unsubscribe-auth/AASDbVGUIK51iOnhLHKg2aWuykW2Y7Imks5utHztgaJpZM4XvMif .

shi-weili commented 5 years ago

@benjaminbojko the line is removed and code is merged!

benjaminbojko commented 5 years ago

Thanks!

On Thu, Nov 8, 2018, 5:00 PM Weili Shi <notifications@github.com wrote:

@benjaminbojko https://github.com/benjaminbojko the line is removed and code is merged!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bluecadet/Cinder-BluecadetViews/pull/134#issuecomment-437171699, or mute the thread https://github.com/notifications/unsubscribe-auth/AASDbfTTwnawkbJpeSoUxGNa20dUQDkgks5utKmIgaJpZM4XvMif .