{"action":"created","issue":{"url":"https://api.github.com/repos/jquery/jquery/issues/2374","labels_url":"https://api.github.com/repos/jquery/jquery/issues/2374/labels{/name}","comments_url":"https://api.github.com/repos/jquery/jquery/issues/2374/comments","events_url":"https://api.github.com/repos/jquery/jquery/issues/2374/events","html_url":"https://github.com/jquery/jquery/pull/2374","id":84166978,"number":2374,"title":"CSS: try more simplified approach to show/hide methods","user":{"login":"markelog","id":945528,"avatar_url":"https://avatars.githubusercontent.com/u/945528?v=3","gravatar_id":"","url":"https://api.github.com/users/markelog","html_url":"https://github.com/markelog","followers_url":"https://api.github.com/users/markelog/followers","following_url":"https://api.github.com/users/markelog/following{/other_user}","gists_url":"https://api.github.com/users/markelog/gists{/gist_id}","starred_url":"https://api.github.com/users/markelog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markelog/subscriptions","organizations_url":"https://api.github.com/users/markelog/orgs","repos_url":"https://api.github.com/users/markelog/repos","events_url":"https://api.github.com/users/markelog/events{/privacy}","received_events_url":"https://api.github.com/users/markelog/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jquery/jquery/labels/CLA:%20Valid","name":"CLA: Valid","color":"007700"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":15,"created_at":"2015-06-02T19:21:28Z","updated_at":"2015-06-03T16:45:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jquery/jquery/pulls/2374","html_url":"https://github.com/jquery/jquery/pull/2374","diff_url":"https://github.com/jquery/jquery/pull/2374.diff","patch_url":"https://github.com/jquery/jquery/pull/2374.patch"},"body":"This is WIP for the feedback.\r\nRef gh-2308\r\n\r\nOkay, this is first option that i mentioned at the meeting.\r\n\r\nBasically, we don't care about \"responsive stylesheets\", detached elements - all that stuff.\r\nWe just take a most simplified and expected approach - default display or `none` that is it.\r\n`show()` - will show element, `hide()` - will hide it.\r\n\r\nIf this is custom or non-standart element - `display: block`, not spec default `display:inline`.\r\n\r\nI remember @paulirish advocated for something like this, if you still interested in this issue, please see the full story <a href=\"https://github.com/jquery/jquery/issues/2308\">here</a>.\r\n\r\n--\r\nBreaking -\r\n\r\n```js\r\n// div { display: inline; }\r\n$( \"div\" ).hide().show(); // block\r\n// or\r\n$( \"div\" ).slideUp(function() {\r\n\t$( this ).slideDown();  // block\r\n});\r\n```\r\n\r\nSince perf issue is only about `show/hide` methods we could preserve above behaviour for the those aliases by dividing render logic (`showHide` method) between simple `show/hide` and animation."},"comment":{"url":"https://api.github.com/repos/jquery/jquery/issues/comments/108519039","html_url":"https://github.com/jquery/jquery/pull/2374#issuecomment-108519039","issue_url":"https://api.github.com/repos/jquery/jquery/issues/2374","id":108519039,"user":{"login":"gibson042","id":1199584,"avatar_url":"https://avatars.githubusercontent.com/u/1199584?v=3","gravatar_id":"","url":"https://api.github.com/users/gibson042","html_url":"https://github.com/gibson042","followers_url":"https://api.github.com/users/gibson042/followers","following_url":"https://api.github.com/users/gibson042/following{/other_user}","gists_url":"https://api.github.com/users/gibson042/gists{/gist_id}","starred_url":"https://api.github.com/users/gibson042/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gibson042/subscriptions","organizations_url":"https://api.github.com/users/gibson042/orgs","repos_url":"https://api.github.com/users/gibson042/repos","events_url":"https://api.github.com/users/gibson042/events{/privacy}","received_events_url":"https://api.github.com/users/gibson042/received_events","type":"User","site_admin":false},"created_at":"2015-06-03T16:45:01Z","updated_at":"2015-06-03T16:45:01Z","body":"Any behavioral distinction between e.g. `.show()` and `.show( 0 )` would be unacceptably confusing and error-prone for users."}}