{"action":"created","issue":{"url":"https://api.github.com/repos/contao/core/issues/7734","labels_url":"https://api.github.com/repos/contao/core/issues/7734/labels{/name}","comments_url":"https://api.github.com/repos/contao/core/issues/7734/comments","events_url":"https://api.github.com/repos/contao/core/issues/7734/events","html_url":"https://github.com/contao/core/issues/7734","id":65276627,"number":7734,"title":"Minify the markup must not remove all spaces, but collapse instead","user":{"login":"Ruudt","id":1218555,"avatar_url":"https://avatars.githubusercontent.com/u/1218555?v=3","gravatar_id":"","url":"https://api.github.com/users/Ruudt","html_url":"https://github.com/Ruudt","followers_url":"https://api.github.com/users/Ruudt/followers","following_url":"https://api.github.com/users/Ruudt/following{/other_user}","gists_url":"https://api.github.com/users/Ruudt/gists{/gist_id}","starred_url":"https://api.github.com/users/Ruudt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ruudt/subscriptions","organizations_url":"https://api.github.com/users/Ruudt/orgs","repos_url":"https://api.github.com/users/Ruudt/repos","events_url":"https://api.github.com/users/Ruudt/events{/privacy}","received_events_url":"https://api.github.com/users/Ruudt/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/contao/core/labels/defect","name":"defect","color":"bd2c00"},{"url":"https://api.github.com/repos/contao/core/labels/up%20for%20discussion","name":"up for discussion","color":"2a982a"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/contao/core/milestones/110","html_url":"https://github.com/contao/core/milestones/3.5.1","labels_url":"https://api.github.com/repos/contao/core/milestones/110/labels","id":1151054,"number":110,"title":"3.5.1","description":"Contao 3.5.1 (bugfix release)","creator":{"login":"leofeyer","id":1192057,"avatar_url":"https://avatars.githubusercontent.com/u/1192057?v=3","gravatar_id":"","url":"https://api.github.com/users/leofeyer","html_url":"https://github.com/leofeyer","followers_url":"https://api.github.com/users/leofeyer/followers","following_url":"https://api.github.com/users/leofeyer/following{/other_user}","gists_url":"https://api.github.com/users/leofeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/leofeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/leofeyer/subscriptions","organizations_url":"https://api.github.com/users/leofeyer/orgs","repos_url":"https://api.github.com/users/leofeyer/repos","events_url":"https://api.github.com/users/leofeyer/events{/privacy}","received_events_url":"https://api.github.com/users/leofeyer/received_events","type":"User","site_admin":false},"open_issues":13,"closed_issues":3,"state":"open","created_at":"2015-06-05T12:19:51Z","updated_at":"2015-07-16T13:29:56Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-03-30T18:50:02Z","updated_at":"2015-07-16T14:47:25Z","closed_at":null,"body":"Contao settings allow to \"Minify the markup\". This removes whitespaces between HTML elements. But that alters the structure of the document. Therefore Contao should collapse whitespaces instead of removing them altogether.\r\n\r\nIn the examples below decrease the width of the \"Result\" pane until the menu doesn't fit. Because the HTML changed the results are different between the two examples. You can see the difference instantly because the spaces are visible in the first example. Tested with Chrome and Firefox.\r\n\r\nExample with whitespaces:\r\nhttps://jsfiddle.net/nkqwbfyo/\r\n\r\nExample without whitespaces:\r\nhttps://jsfiddle.net/nkqwbfyo/1/"},"comment":{"url":"https://api.github.com/repos/contao/core/issues/comments/121978578","html_url":"https://github.com/contao/core/issues/7734#issuecomment-121978578","issue_url":"https://api.github.com/repos/contao/core/issues/7734","id":121978578,"user":{"login":"leofeyer","id":1192057,"avatar_url":"https://avatars.githubusercontent.com/u/1192057?v=3","gravatar_id":"","url":"https://api.github.com/users/leofeyer","html_url":"https://github.com/leofeyer","followers_url":"https://api.github.com/users/leofeyer/followers","following_url":"https://api.github.com/users/leofeyer/following{/other_user}","gists_url":"https://api.github.com/users/leofeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/leofeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/leofeyer/subscriptions","organizations_url":"https://api.github.com/users/leofeyer/orgs","repos_url":"https://api.github.com/users/leofeyer/repos","events_url":"https://api.github.com/users/leofeyer/events{/privacy}","received_events_url":"https://api.github.com/users/leofeyer/received_events","type":"User","site_admin":false},"created_at":"2015-07-16T14:47:25Z","updated_at":"2015-07-16T14:47:25Z","body":"As discussed on Mumble on July 16th, we should only strip leading whitespace characters like the Google pagespeed module does. And we should apply the above fix to all inline elements."}}