{"action":"created","issue":{"url":"https://api.github.com/repos/kanaka/noVNC/issues/446","labels_url":"https://api.github.com/repos/kanaka/noVNC/issues/446/labels{/name}","comments_url":"https://api.github.com/repos/kanaka/noVNC/issues/446/comments","events_url":"https://api.github.com/repos/kanaka/noVNC/issues/446/events","html_url":"https://github.com/kanaka/noVNC/issues/446","id":57307415,"number":446,"title":"Please, no getElementById within the code","user":{"login":"ibc","id":16191,"avatar_url":"https://avatars.githubusercontent.com/u/16191?v=3","gravatar_id":"","url":"https://api.github.com/users/ibc","html_url":"https://github.com/ibc","followers_url":"https://api.github.com/users/ibc/followers","following_url":"https://api.github.com/users/ibc/following{/other_user}","gists_url":"https://api.github.com/users/ibc/gists{/gist_id}","starred_url":"https://api.github.com/users/ibc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ibc/subscriptions","organizations_url":"https://api.github.com/users/ibc/orgs","repos_url":"https://api.github.com/users/ibc/repos","events_url":"https://api.github.com/users/ibc/events{/privacy}","received_events_url":"https://api.github.com/users/ibc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-11T12:02:10Z","updated_at":"2015-02-11T12:20:58Z","closed_at":null,"body":"https://github.com/kanaka/noVNC/blob/master/include/display.js#L229\r\n\r\n```\r\nvar cb = document.getElementById('noVNC-control-bar');\r\n```\r\n\r\nNo please, really not. Don't force me to have a specific HTML element with \"id=noVNC-control-bar\". Please not. Yo have added that because ui.js and base.css use such an HTML element, but that is just the demo application, not the library. Please don't mix them.\r\n\r\nConsider the case in which I want to have two isolated and 100% separated noVNC displays in the same web app.\r\n\r\nPlease, don't do that.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/kanaka/noVNC/issues/comments/73872760","html_url":"https://github.com/kanaka/noVNC/issues/446#issuecomment-73872760","issue_url":"https://api.github.com/repos/kanaka/noVNC/issues/446","id":73872760,"user":{"login":"ibc","id":16191,"avatar_url":"https://avatars.githubusercontent.com/u/16191?v=3","gravatar_id":"","url":"https://api.github.com/users/ibc","html_url":"https://github.com/ibc","followers_url":"https://api.github.com/users/ibc/followers","following_url":"https://api.github.com/users/ibc/following{/other_user}","gists_url":"https://api.github.com/users/ibc/gists{/gist_id}","starred_url":"https://api.github.com/users/ibc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ibc/subscriptions","organizations_url":"https://api.github.com/users/ibc/orgs","repos_url":"https://api.github.com/users/ibc/repos","events_url":"https://api.github.com/users/ibc/events{/privacy}","received_events_url":"https://api.github.com/users/ibc/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T12:20:58Z","updated_at":"2015-02-11T12:20:58Z","body":"Note that `RFB` requires `Display`, so at the end `Display` is part of the core \"library\". If it needs to have a reference to an HTML element then let it to be passed via API/configuration."}}