{"action":"created","issue":{"url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/15","labels_url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/15/labels{/name}","comments_url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/15/comments","events_url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/15/events","html_url":"https://github.com/jonom/silverstripe-betternavigator/pull/15","id":56957578,"number":15,"title":"Ie8 compat","user":{"login":"anselmdk","id":1316533,"avatar_url":"https://avatars.githubusercontent.com/u/1316533?v=3","gravatar_id":"","url":"https://api.github.com/users/anselmdk","html_url":"https://github.com/anselmdk","followers_url":"https://api.github.com/users/anselmdk/followers","following_url":"https://api.github.com/users/anselmdk/following{/other_user}","gists_url":"https://api.github.com/users/anselmdk/gists{/gist_id}","starred_url":"https://api.github.com/users/anselmdk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anselmdk/subscriptions","organizations_url":"https://api.github.com/users/anselmdk/orgs","repos_url":"https://api.github.com/users/anselmdk/repos","events_url":"https://api.github.com/users/anselmdk/events{/privacy}","received_events_url":"https://api.github.com/users/anselmdk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-08T16:37:22Z","updated_at":"2015-02-17T23:07:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/pulls/15","html_url":"https://github.com/jonom/silverstripe-betternavigator/pull/15","diff_url":"https://github.com/jonom/silverstripe-betternavigator/pull/15.diff","patch_url":"https://github.com/jonom/silverstripe-betternavigator/pull/15.patch"},"body":"I'm was just testing my site in IE8, and ran into the issue that betternavigator breaks the JS in IE8, I'd prefer if we could just remove betternavigator entirely when dealing with IE8.\r\nSee my suggestion here."},"comment":{"url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/comments/74775124","html_url":"https://github.com/jonom/silverstripe-betternavigator/pull/15#issuecomment-74775124","issue_url":"https://api.github.com/repos/jonom/silverstripe-betternavigator/issues/15","id":74775124,"user":{"login":"jonom","id":1079425,"avatar_url":"https://avatars.githubusercontent.com/u/1079425?v=3","gravatar_id":"","url":"https://api.github.com/users/jonom","html_url":"https://github.com/jonom","followers_url":"https://api.github.com/users/jonom/followers","following_url":"https://api.github.com/users/jonom/following{/other_user}","gists_url":"https://api.github.com/users/jonom/gists{/gist_id}","starred_url":"https://api.github.com/users/jonom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jonom/subscriptions","organizations_url":"https://api.github.com/users/jonom/orgs","repos_url":"https://api.github.com/users/jonom/repos","events_url":"https://api.github.com/users/jonom/events{/privacy}","received_events_url":"https://api.github.com/users/jonom/received_events","type":"User","site_admin":false},"created_at":"2015-02-17T23:07:16Z","updated_at":"2015-02-17T23:07:16Z","body":"Hi @anselmdk thanks for the feedback and PR, sorry for the slow reply. I would prefer if the menu was available in IE8 but we just lose the collapse/expand functionality. If that's okay with you would you mind altering your PR to remove the `else` section (keep the `if`) and change the default class of the `#BetterNavigator` div to `open` instead of `collapsed` in BetterNavigator.ss. I think that should do it. Then if you could squash the commits that would be great.\r\n\r\nOh, would also need to change\r\n```\r\n\t//Restore menu state\r\n\tif (getCookie('BetterNavigator') === 'open') {\r\n\t\tBetterNavigator.className = 'open';\r\n\t}\r\n```\r\nto \r\n```\r\n\t//Restore menu state\r\n\tif (getCookie('BetterNavigator') === 'collapsed') {\r\n\t\tBetterNavigator.className = 'collapsed';\r\n\t}\r\n```\r\nIf you're not available to make these changes let me know and I can implement them."}}