{"comment":{"created_at":"2011-10-10T12:44:50Z","body":"I believe this has been deployed so closing.","updated_at":"2011-10-10T12:44:50Z","url":"https://api.github.com/repos/TiddlySpace/tiddlyspace/issues/comments/2346971","id":2346971,"user":{"avatar_url":"https://secure.gravatar.com/avatar/45b0863bddc42c3bbb08284d1863ad95?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/jdlrobson","id":148752,"login":"jdlrobson"}},"action":"created","issue":{"number":767,"created_at":"2011-09-26T17:17:52Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"The general rule of thumb is that you should not use px in CSS in\nwhich you have any expectation that people may wish to change their\nbrowser font size.\n\nIf px are used for the width of #container, then increasing the \nfont size means the text must wrap far too soon.\n\nI would argue that the correct for controlling the width of \nthe container element is em, perhaps 50.\n\nThen related elements, such as padding should be similarly scaled using\nem, and the presentation will flow and scale correctly as fonts are scaled.\n\nSee also comments elsewhere about how using px means that there is vast\nunused real estate when viewing the default HTML rep on a large screen.","comments":3,"title":"Default CSS uses px to control text display width and other sizes.","updated_at":"2011-10-10T12:44:50Z","url":"https://api.github.com/repos/TiddlySpace/tiddlyspace/issues/767","id":1741706,"assignee":null,"milestone":null,"closed_at":"2011-10-10T12:44:50Z","user":{"avatar_url":"https://secure.gravatar.com/avatar/0c8a56d012e5a2cabe00f28b5f68ce83?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/cdent","id":34498,"login":"cdent"},"html_url":"https://github.com/TiddlySpace/tiddlyspace/issues/767","labels":[],"state":"closed"},"legacy":{"issue_id":1741706,"comment_id":2346971}}