{"action":"closed","issue":{"url":"https://api.github.com/repos/bgrins/TinyColor/issues/77","labels_url":"https://api.github.com/repos/bgrins/TinyColor/issues/77/labels{/name}","comments_url":"https://api.github.com/repos/bgrins/TinyColor/issues/77/comments","events_url":"https://api.github.com/repos/bgrins/TinyColor/issues/77/events","html_url":"https://github.com/bgrins/TinyColor/issues/77","id":54279153,"number":77,"title":"Potential toString vs getOriginalInput confusion ","user":{"login":"le717","id":3382464,"avatar_url":"https://avatars.githubusercontent.com/u/3382464?v=3","gravatar_id":"","url":"https://api.github.com/users/le717","html_url":"https://github.com/le717","followers_url":"https://api.github.com/users/le717/followers","following_url":"https://api.github.com/users/le717/following{/other_user}","gists_url":"https://api.github.com/users/le717/gists{/gist_id}","starred_url":"https://api.github.com/users/le717/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/le717/subscriptions","organizations_url":"https://api.github.com/users/le717/orgs","repos_url":"https://api.github.com/users/le717/repos","events_url":"https://api.github.com/users/le717/events{/privacy}","received_events_url":"https://api.github.com/users/le717/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-01-14T03:10:09Z","updated_at":"2015-01-16T23:07:58Z","closed_at":"2015-01-16T23:07:58Z","body":"In having adobe/brackets#9596 reviewed, @redmunds brought out an interesting observation:\r\n\r\n> (Me) `tinycolor.toString()` actually works as the author intended. It does not ignore the original case, but rather normalizes (to lowercase) it before returning it. There was actually no way to get the original input (it wasn't even stored) until I suggested it and sent a patch in the form of `tinycolor.getOriginalInput()`.\r\n\r\n> Yes, but since you added `tinycolor.getOriginalInput()`, I think it changes expectations. I would expect that method to be called something like `toNormalizedString()`.\r\n\r\nI am wondering what your thoughts are on this, @bgrins. I know `toString()` is a long-standing method and removing/renaming it would be a huge breaking change, but would it be possible to better document the normalization it performs and/or rename `getOriginalInput()` to something that better describes it's actions vs `toString()`? I mention renaming as it is a very recent edition to TinyColor and probably isn't used much yet. \r\n\r\n"}}