{"action":"created","issue":{"url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/51","labels_url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/51/labels{/name}","comments_url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/51/comments","events_url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/51/events","html_url":"https://github.com/imagej/imagej-ui-swing/pull/51","id":100517978,"number":51,"title":"Cleanup Script Editor","user":{"login":"Squareys","id":1889284,"avatar_url":"https://avatars.githubusercontent.com/u/1889284?v=3","gravatar_id":"","url":"https://api.github.com/users/Squareys","html_url":"https://github.com/Squareys","followers_url":"https://api.github.com/users/Squareys/followers","following_url":"https://api.github.com/users/Squareys/following{/other_user}","gists_url":"https://api.github.com/users/Squareys/gists{/gist_id}","starred_url":"https://api.github.com/users/Squareys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Squareys/subscriptions","organizations_url":"https://api.github.com/users/Squareys/orgs","repos_url":"https://api.github.com/users/Squareys/repos","events_url":"https://api.github.com/users/Squareys/events{/privacy}","received_events_url":"https://api.github.com/users/Squareys/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-12T10:53:51Z","updated_at":"2015-08-12T16:05:18Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/imagej/imagej-ui-swing/pulls/51","html_url":"https://github.com/imagej/imagej-ui-swing/pull/51","diff_url":"https://github.com/imagej/imagej-ui-swing/pull/51.diff","patch_url":"https://github.com/imagej/imagej-ui-swing/pull/51.patch"},"body":"Hello everybody, \r\n\r\nas part of #40, I cleaned up the existing script editor code, extracted some classes for readability and added a bunch of javadoc. This involved code formatting, which I tried to keep in separate commits. Everything is done in small and (hopefully) clear steps.\r\n\r\nIn general this is all refactoring and should not result in any visible changes for the user, but does break the API every now and then (changing member access to private and providing a getter for example).\r\n\r\nGreetings, \r\nSquareys"},"comment":{"url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/comments/130356538","html_url":"https://github.com/imagej/imagej-ui-swing/pull/51#issuecomment-130356538","issue_url":"https://api.github.com/repos/imagej/imagej-ui-swing/issues/51","id":130356538,"user":{"login":"Squareys","id":1889284,"avatar_url":"https://avatars.githubusercontent.com/u/1889284?v=3","gravatar_id":"","url":"https://api.github.com/users/Squareys","html_url":"https://github.com/Squareys","followers_url":"https://api.github.com/users/Squareys/followers","following_url":"https://api.github.com/users/Squareys/following{/other_user}","gists_url":"https://api.github.com/users/Squareys/gists{/gist_id}","starred_url":"https://api.github.com/users/Squareys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Squareys/subscriptions","organizations_url":"https://api.github.com/users/Squareys/orgs","repos_url":"https://api.github.com/users/Squareys/repos","events_url":"https://api.github.com/users/Squareys/events{/privacy}","received_events_url":"https://api.github.com/users/Squareys/received_events","type":"User","site_admin":false},"created_at":"2015-08-12T16:05:18Z","updated_at":"2015-08-12T16:05:18Z","body":"What if the `setTitle` does `frame.setTitle()`? It cannot be considered good stlye to have a method for that. Since it was synchronized, that is different though."}}