{"action":"opened","issue":{"url":"https://api.github.com/repos/ryanisaacg/quicksilver/issues/315","repository_url":"https://api.github.com/repos/ryanisaacg/quicksilver","labels_url":"https://api.github.com/repos/ryanisaacg/quicksilver/issues/315/labels{/name}","comments_url":"https://api.github.com/repos/ryanisaacg/quicksilver/issues/315/comments","events_url":"https://api.github.com/repos/ryanisaacg/quicksilver/issues/315/events","html_url":"https://github.com/ryanisaacg/quicksilver/issues/315","id":341489579,"node_id":"MDU6SXNzdWUzNDE0ODk1Nzk=","number":315,"title":"window::draw_ex()","user":{"login":"Xoronic","id":30958364,"node_id":"MDQ6VXNlcjMwOTU4MzY0","avatar_url":"https://avatars0.githubusercontent.com/u/30958364?v=4","gravatar_id":"","url":"https://api.github.com/users/Xoronic","html_url":"https://github.com/Xoronic","followers_url":"https://api.github.com/users/Xoronic/followers","following_url":"https://api.github.com/users/Xoronic/following{/other_user}","gists_url":"https://api.github.com/users/Xoronic/gists{/gist_id}","starred_url":"https://api.github.com/users/Xoronic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Xoronic/subscriptions","organizations_url":"https://api.github.com/users/Xoronic/orgs","repos_url":"https://api.github.com/users/Xoronic/repos","events_url":"https://api.github.com/users/Xoronic/events{/privacy}","received_events_url":"https://api.github.com/users/Xoronic/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-16T11:57:14Z","updated_at":"2018-07-16T11:57:14Z","closed_at":null,"author_association":"NONE","body":"I just cannot figure out why it is called draw_ex and why it is different from `draw` and `draw_color`..\r\n\r\nFrom looking at the implementation of the function, I can tell that it has an extra z parameter? *But what does `ex` mean?*\r\n\r\n## 3 Suggestions\r\n\r\n### 1: Change the documentation for `draw`, `draw_color`, `draw_ex` and `draw_params` functions\r\nRight now, all of the draw functions have the same documentation: \r\n\r\n```\r\n    /// Draw a single object to the screen\r\n    ///\r\n    /// It will not appear until Window::flush is called\r\n```\r\n\r\nIf multiple functions have the exact same documentation, the documentation loses its worth.\r\n\r\n### 2: Rename `draw_ex` to something that is clear\r\nI have no idea what `ex` stands for, even after looking at the implementation. I get that function names should be not too long to type, but `draw_ex` is *too* short.\r\n\r\n### 3: Changing the `draw` function\r\nLeave transformation for specialized draw-functions, and drop it for the simplest one. All shapes already have a position, don't they? At least `Rectangle`, `Circle` and `Line` do have positions."}}
{"actor":{"display_login":"Xoronic"}}