{"action":"created","issue":{"url":"https://api.github.com/repos/orf/HtmlToWord/issues/7","labels_url":"https://api.github.com/repos/orf/HtmlToWord/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/orf/HtmlToWord/issues/7/comments","events_url":"https://api.github.com/repos/orf/HtmlToWord/issues/7/events","html_url":"https://github.com/orf/HtmlToWord/pull/7","id":57294216,"number":7,"title":"Add support for inline css styles: font-size, color; and map the 'class' attribute to word styles","user":{"login":"dgaus","id":3506077,"avatar_url":"https://avatars.githubusercontent.com/u/3506077?v=3","gravatar_id":"","url":"https://api.github.com/users/dgaus","html_url":"https://github.com/dgaus","followers_url":"https://api.github.com/users/dgaus/followers","following_url":"https://api.github.com/users/dgaus/following{/other_user}","gists_url":"https://api.github.com/users/dgaus/gists{/gist_id}","starred_url":"https://api.github.com/users/dgaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgaus/subscriptions","organizations_url":"https://api.github.com/users/dgaus/orgs","repos_url":"https://api.github.com/users/dgaus/repos","events_url":"https://api.github.com/users/dgaus/events{/privacy}","received_events_url":"https://api.github.com/users/dgaus/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-11T09:42:43Z","updated_at":"2015-02-11T20:30:06Z","closed_at":"2015-02-11T20:30:02Z","pull_request":{"url":"https://api.github.com/repos/orf/HtmlToWord/pulls/7","html_url":"https://github.com/orf/HtmlToWord/pull/7","diff_url":"https://github.com/orf/HtmlToWord/pull/7.diff","patch_url":"https://github.com/orf/HtmlToWord/pull/7.patch"},"body":"Hi, this is a first attempt at implementing inline style support. We had to change span to inherit from BaseElement instead of IgnoredElement, because redactor and other editors apply styles by wrapping elements in a span, and otherwise it got ignored. It seemed to have no ill effects, but if we are missing something please let us know.\r\nWe also added a mapping from css 'class' to word style, since it is conceptually is very similar, and we needed to apply these to individual elements. The restriction would be that only one 'class' is supported per element, but this is a word restriction.\r\nColors are only accepted in rgb(x,x,x) format currently, since it's what redactor outputs, and font size only in pixels, it could be easily adapted to be more flexible, but we wanted to share this approach before putting in any more effort in this. Thanks!"},"comment":{"url":"https://api.github.com/repos/orf/HtmlToWord/issues/comments/73958462","html_url":"https://github.com/orf/HtmlToWord/pull/7#issuecomment-73958462","issue_url":"https://api.github.com/repos/orf/HtmlToWord/issues/7","id":73958462,"user":{"login":"orf","id":1027207,"avatar_url":"https://avatars.githubusercontent.com/u/1027207?v=3","gravatar_id":"","url":"https://api.github.com/users/orf","html_url":"https://github.com/orf","followers_url":"https://api.github.com/users/orf/followers","following_url":"https://api.github.com/users/orf/following{/other_user}","gists_url":"https://api.github.com/users/orf/gists{/gist_id}","starred_url":"https://api.github.com/users/orf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/orf/subscriptions","organizations_url":"https://api.github.com/users/orf/orgs","repos_url":"https://api.github.com/users/orf/repos","events_url":"https://api.github.com/users/orf/events{/privacy}","received_events_url":"https://api.github.com/users/orf/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T20:30:06Z","updated_at":"2015-02-11T20:30:06Z","body":"Thanks for this! I am going to merge it but not push a release to PyPi (unless it would help you if I did) because I'm thinking about re-writing the library. At the moment it is too tied to HTML, I want to remove HTML-specific elements (like span and bold) and replace them with with a more abstract set of classes representing changes to the document (so in this case maybe a Style class). You've opened a few pull requests so I wondered if you had any thoughts on this?"}}