miketheprogrammer / go-thrust

Cross Platform UI Kit powered by Blink/V8/Chromium Content Lib
MIT License
446 stars 34 forks source link

Priority: Restructure Repository #32

Open miketheprogrammer opened 9 years ago

miketheprogrammer commented 9 years ago

Move (window, menu, session ) => src/bindings/ Expose NewWindow NewMenu NewSession via package Thrust Figure out what the best option is for Spawn, Connection, Commands, Dispatcher packages. Start giving thought to public vs private api's. Remove public directory, public directories should be tutorial and example local Remove makefile, its stupid to have a platform dependent file. Remove package Common, its useless, and in removing it, remove alexcasaro log, lets just use standard logger.

--- my thoughts --- Spawn should be moved to package Thrust.

Thrust.RunCore() instead of spawn.Run
Thrust.NewWindow() instead of window.NewWindow
Thrust.NewSession() instead of session.NewSession
Thrust.NewMenu()  instead of menu.NewMenu
Thrust.RunDispatcher() instead of Dispatcher.RunLoop()

Are we bulking to much into one package, i still think src/bindings/window will have its own NewWindow method, however that is purely to resolve any cyclic issues we may encounter down the road.

@tehbilly @gerred Feel free to chime in

spolu commented 9 years ago

:+1:

miketheprogrammer commented 9 years ago

Im hacking this out on branch refactor_4.0

miketheprogrammer commented 9 years ago

Done. Merged Into Master

miketheprogrammer commented 9 years ago

Can someone work on updating the tutorials. I just updated the Examples.