Skip to content

Conversation

@arieljake
Copy link

Hi, I would like to offer a contribution to Nide: an editor for package.json files.

It was built in the nature of the other editors in the app.

I do have a lingering question about sync'ing files between editors.

@arieljake arieljake closed this Jul 8, 2012
@coreh coreh reopened this Jul 8, 2012
@coreh
Copy link
Member

coreh commented Jul 8, 2012

Hey @arieljake, is there a reason why this was closed? do you want me to wait for something before merging it?

@coreh
Copy link
Member

coreh commented Jul 8, 2012

BTW, thanks for the patch :-D

@arieljake
Copy link
Author

no reason other than that I wanted to stop new commits from being added. This is my first pull request so im trying to figure out how to limit the commits i put into it.

… added show-line-numbers and indent-unit

preferences dialog auto saves :input fields as cookies by the field
'name'
preferences dialog, on saving line numbers pref, updates code editors:
ui -> editor pool -> editors
preferences dialog shows/hides but does not rebuild each time anymore

NOTE: all editor types return "this" from constructor not just DIV
element anymore, and to preserve backwards-compatibility, sets the root
DIV as 'this.element'
@arieljake
Copy link
Author

to help with handling pull requests, should we prioritize creating some tests?
have you used zombie before?

@arieljake
Copy link
Author

also, if you do accept the additions, can you review my CSS styling and adjust as necessary to keep Nide beautiful! :)

arieljake added 2 commits July 9, 2012 00:40
gear menu and code runner using new base classes
beginning of code autocompletion
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.

2 participants