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.

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

Extract components #93

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

ghostandthemachine
Copy link

Updates

  • separated layout from application functions
    • core.clj now contains just layout, adding behaviors, and initialization
    • dev_tools.clj contains all of the non-layout and view oriented functions
  • separated component initialization in (create-app) into a few core initialization functions
    • editor (make-text-editor-comp app-init)
    • file-tree (make-file-tree-comp app-init)
    • repl (make-repl-comp app-init)
    • doc-view (make-doc-view-comp app-init)
    • doc-nav (make-doc-nav-comp app-init)
  • moved adding component handlers from create-app to (add-behaviors) allowing (create-app) to deal only with view layout
  • update main to use seesaw show! as well as invoke-later
  • gone native!
    • added seesaw call to (native!) in brackets.clj. This isn't a logical place to put it but seesaw require native! to be called before any other seesaw calls and this is the first included file in dev_tools.clj
    • added :jvm-opts to project.clj which load "Clooj" menu instead of clojure.main in OSX menubar

Known Issues

  • still get a lot of errors for Don't know how to create ISeq from: clojure.lang.Symbol which is caused by clooj.help$ns_available_names.invoke(help.clj:114). I get these errors when I clone the current clooj repo and run it too though so I don't think the updates have created this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant