{"action":"created","issue":{"url":"https://api.github.com/repos/emacs-ess/ESS/issues/217","labels_url":"https://api.github.com/repos/emacs-ess/ESS/issues/217/labels{/name}","comments_url":"https://api.github.com/repos/emacs-ess/ESS/issues/217/comments","events_url":"https://api.github.com/repos/emacs-ess/ESS/issues/217/events","html_url":"https://github.com/emacs-ess/ESS/issues/217","id":100739679,"number":217,"title":"Tidy up indentation styles","user":{"login":"lionel-","id":4465050,"avatar_url":"https://avatars.githubusercontent.com/u/4465050?v=3","gravatar_id":"","url":"https://api.github.com/users/lionel-","html_url":"https://github.com/lionel-","followers_url":"https://api.github.com/users/lionel-/followers","following_url":"https://api.github.com/users/lionel-/following{/other_user}","gists_url":"https://api.github.com/users/lionel-/gists{/gist_id}","starred_url":"https://api.github.com/users/lionel-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lionel-/subscriptions","organizations_url":"https://api.github.com/users/lionel-/orgs","repos_url":"https://api.github.com/users/lionel-/repos","events_url":"https://api.github.com/users/lionel-/events{/privacy}","received_events_url":"https://api.github.com/users/lionel-/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-08-13T10:59:13Z","updated_at":"2015-08-19T22:26:05Z","closed_at":null,"body":"The next ESS version will feature a major redesign of the indentation engine and settings and I think we should take this opportunity to simplify the default styles. They are currently a bit messy because:\r\n\r\n- most of the styles are quite similar;\r\n- the names of the styles now seem a bit arbitrary. It is not clear why the C++ or GNU styles are called like that for example.\r\n\r\nI think we could narrow them down to 3 styles:\r\n\r\n- RRR, which could be renamed to ESS\r\n- RStudio, which would provide compatibility with a major editor\r\n- A \"hanging\" style. Basically the RRR settings with `ess-offset-block` set to `open-delim`. Compared to RRR, this style would not try to save horizontal space. E.g.:\r\n\r\n```r\r\n# Instead of\r\nobject <- lapply(object, function(x) {\r\n    body\r\n},\r\nargument)\r\n\r\n# The \"hanging\" or \"stretched\" style would yield\r\nobject <- lapply(object, function(x) {\r\n                             body\r\n                         },\r\n                 argument)\r\n```\r\n\r\nThe documentation could provide the deprecated styles as a reference for users who were relying on them."},"comment":{"url":"https://api.github.com/repos/emacs-ess/ESS/issues/comments/132806837","html_url":"https://github.com/emacs-ess/ESS/issues/217#issuecomment-132806837","issue_url":"https://api.github.com/repos/emacs-ess/ESS/issues/217","id":132806837,"user":{"login":"vspinu","id":1363467,"avatar_url":"https://avatars.githubusercontent.com/u/1363467?v=3","gravatar_id":"","url":"https://api.github.com/users/vspinu","html_url":"https://github.com/vspinu","followers_url":"https://api.github.com/users/vspinu/followers","following_url":"https://api.github.com/users/vspinu/following{/other_user}","gists_url":"https://api.github.com/users/vspinu/gists{/gist_id}","starred_url":"https://api.github.com/users/vspinu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vspinu/subscriptions","organizations_url":"https://api.github.com/users/vspinu/orgs","repos_url":"https://api.github.com/users/vspinu/repos","events_url":"https://api.github.com/users/vspinu/events{/privacy}","received_events_url":"https://api.github.com/users/vspinu/received_events","type":"User","site_admin":false},"created_at":"2015-08-19T22:26:05Z","updated_at":"2015-08-19T22:26:05Z","body":"Yeh. Having the DEFAULT being the \"default\" would mean either changing it or adopting it. No way we can keep backward compatibility on this one unless we leave it as it is (aka picking the default values from globals). \r\n\r\nAs to collaboration, R-Studio style should do the job in virtually all cases. "}}