{"action":"created","issue":{"url":"https://api.github.com/repos/pixijs/pixi.js/issues/2539","repository_url":"https://api.github.com/repos/pixijs/pixi.js","labels_url":"https://api.github.com/repos/pixijs/pixi.js/issues/2539/labels{/name}","comments_url":"https://api.github.com/repos/pixijs/pixi.js/issues/2539/comments","events_url":"https://api.github.com/repos/pixijs/pixi.js/issues/2539/events","html_url":"https://github.com/pixijs/pixi.js/pull/2539","id":151828570,"number":2539,"title":"Text: Splitting font setting into it's various style components","user":{"login":"themoonrat","id":1528426,"avatar_url":"https://avatars.githubusercontent.com/u/1528426?v=3","gravatar_id":"","url":"https://api.github.com/users/themoonrat","html_url":"https://github.com/themoonrat","followers_url":"https://api.github.com/users/themoonrat/followers","following_url":"https://api.github.com/users/themoonrat/following{/other_user}","gists_url":"https://api.github.com/users/themoonrat/gists{/gist_id}","starred_url":"https://api.github.com/users/themoonrat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/themoonrat/subscriptions","organizations_url":"https://api.github.com/users/themoonrat/orgs","repos_url":"https://api.github.com/users/themoonrat/repos","events_url":"https://api.github.com/users/themoonrat/events{/privacy}","received_events_url":"https://api.github.com/users/themoonrat/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-04-29T09:40:44Z","updated_at":"2016-05-02T16:18:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pixijs/pixi.js/pulls/2539","html_url":"https://github.com/pixijs/pixi.js/pull/2539","diff_url":"https://github.com/pixijs/pixi.js/pull/2539.diff","patch_url":"https://github.com/pixijs/pixi.js/pull/2539.patch"},"body":"Instead of just setting font, and having to put the settings for bold / size etc. in the right order, these style settings are split up into the 5 style components, which are joined together automatically for you.\r\nThere is now:\r\nfontFamily\r\nfontSize\r\nfontStyle\r\nfontVariant\r\nfontWeight\r\n\r\nBefore any merge is done... I was wondering if the defaults should be changed. I have just taken the 'bold 20pt Arial' and taken those settings as the default. However, because a developer isn't forced to override all style settings for each font they want anymore....\r\n\r\n1. I do not think 'bold' should be enabled by default.\r\n2. I think that the default size should move from 'pt' to an equivilant 'px'. 'px' is easier to understand in a rendering to a canvas that is made up of pixels context.\r\n\r\nThoughts?"},"comment":{"url":"https://api.github.com/repos/pixijs/pixi.js/issues/comments/216281282","html_url":"https://github.com/pixijs/pixi.js/pull/2539#issuecomment-216281282","issue_url":"https://api.github.com/repos/pixijs/pixi.js/issues/2539","id":216281282,"user":{"login":"bigtimebuddy","id":864393,"avatar_url":"https://avatars.githubusercontent.com/u/864393?v=3","gravatar_id":"","url":"https://api.github.com/users/bigtimebuddy","html_url":"https://github.com/bigtimebuddy","followers_url":"https://api.github.com/users/bigtimebuddy/followers","following_url":"https://api.github.com/users/bigtimebuddy/following{/other_user}","gists_url":"https://api.github.com/users/bigtimebuddy/gists{/gist_id}","starred_url":"https://api.github.com/users/bigtimebuddy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bigtimebuddy/subscriptions","organizations_url":"https://api.github.com/users/bigtimebuddy/orgs","repos_url":"https://api.github.com/users/bigtimebuddy/repos","events_url":"https://api.github.com/users/bigtimebuddy/events{/privacy}","received_events_url":"https://api.github.com/users/bigtimebuddy/received_events","type":"User","site_admin":false},"created_at":"2016-05-02T16:18:04Z","updated_at":"2016-05-02T16:18:04Z","body":"Could you add some `TextStyle.font` deprecation code to **src/deprecation.js** and parse the font string as the arguments mentioned here? This would provide better upgrading support for developers.  "}}