{"action":"created","issue":{"url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/3","labels_url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/3/comments","events_url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/3/events","html_url":"https://github.com/propertyshelf/ps.plone.realestatefont/issues/3","id":98941830,"number":3,"title":"Use better class names","user":{"login":"tmassman","id":1116199,"avatar_url":"https://avatars.githubusercontent.com/u/1116199?v=3","gravatar_id":"","url":"https://api.github.com/users/tmassman","html_url":"https://github.com/tmassman","followers_url":"https://api.github.com/users/tmassman/followers","following_url":"https://api.github.com/users/tmassman/following{/other_user}","gists_url":"https://api.github.com/users/tmassman/gists{/gist_id}","starred_url":"https://api.github.com/users/tmassman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmassman/subscriptions","organizations_url":"https://api.github.com/users/tmassman/orgs","repos_url":"https://api.github.com/users/tmassman/repos","events_url":"https://api.github.com/users/tmassman/events{/privacy}","received_events_url":"https://api.github.com/users/tmassman/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"tmassman","id":1116199,"avatar_url":"https://avatars.githubusercontent.com/u/1116199?v=3","gravatar_id":"","url":"https://api.github.com/users/tmassman","html_url":"https://github.com/tmassman","followers_url":"https://api.github.com/users/tmassman/followers","following_url":"https://api.github.com/users/tmassman/following{/other_user}","gists_url":"https://api.github.com/users/tmassman/gists{/gist_id}","starred_url":"https://api.github.com/users/tmassman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmassman/subscriptions","organizations_url":"https://api.github.com/users/tmassman/orgs","repos_url":"https://api.github.com/users/tmassman/repos","events_url":"https://api.github.com/users/tmassman/events{/privacy}","received_events_url":"https://api.github.com/users/tmassman/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/milestones/1","html_url":"https://github.com/propertyshelf/ps.plone.realestatefont/milestones/0.3%20Framework%20independent%20font%20resources","labels_url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/milestones/1/labels","id":1239966,"number":1,"title":"0.3 Framework independent font resources","description":"Font definitions and styles available for Zope and Plone.","creator":{"login":"tmassman","id":1116199,"avatar_url":"https://avatars.githubusercontent.com/u/1116199?v=3","gravatar_id":"","url":"https://api.github.com/users/tmassman","html_url":"https://github.com/tmassman","followers_url":"https://api.github.com/users/tmassman/followers","following_url":"https://api.github.com/users/tmassman/following{/other_user}","gists_url":"https://api.github.com/users/tmassman/gists{/gist_id}","starred_url":"https://api.github.com/users/tmassman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmassman/subscriptions","organizations_url":"https://api.github.com/users/tmassman/orgs","repos_url":"https://api.github.com/users/tmassman/repos","events_url":"https://api.github.com/users/tmassman/events{/privacy}","received_events_url":"https://api.github.com/users/tmassman/received_events","type":"User","site_admin":false},"open_issues":3,"closed_issues":0,"state":"open","created_at":"2015-08-04T10:54:23Z","updated_at":"2015-08-04T10:58:30Z","due_on":"2015-08-13T22:00:00Z","closed_at":null},"comments":1,"created_at":"2015-08-04T10:51:46Z","updated_at":"2015-08-04T12:06:10Z","closed_at":null,"body":"Currently the used prefix is **icon_re** which is not very meaningful. Instead, there should be a prefix of **psf** (for _Propertyshelf Font_), followed by **psf-realestate-[icon name]**. The class names in HTML would than look like this:\r\n\r\n```html\r\n<i class=\"psf psf-realestate-airport\"></i>\r\n```\r\n\r\nThis results in a slightly more \"complex\" markup (or more markup because more characters are used), but it's way more readable and meaningful."},"comment":{"url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/comments/127578494","html_url":"https://github.com/propertyshelf/ps.plone.realestatefont/issues/3#issuecomment-127578494","issue_url":"https://api.github.com/repos/propertyshelf/ps.plone.realestatefont/issues/3","id":127578494,"user":{"login":"Solomonic","id":3539915,"avatar_url":"https://avatars.githubusercontent.com/u/3539915?v=3","gravatar_id":"","url":"https://api.github.com/users/Solomonic","html_url":"https://github.com/Solomonic","followers_url":"https://api.github.com/users/Solomonic/followers","following_url":"https://api.github.com/users/Solomonic/following{/other_user}","gists_url":"https://api.github.com/users/Solomonic/gists{/gist_id}","starred_url":"https://api.github.com/users/Solomonic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Solomonic/subscriptions","organizations_url":"https://api.github.com/users/Solomonic/orgs","repos_url":"https://api.github.com/users/Solomonic/repos","events_url":"https://api.github.com/users/Solomonic/events{/privacy}","received_events_url":"https://api.github.com/users/Solomonic/received_events","type":"User","site_admin":false},"created_at":"2015-08-04T12:06:10Z","updated_at":"2015-08-04T12:06:10Z","body":"The standard solution we would like to use  is:\r\n```html\r\n<i class=\"icon icon-airport\"></i>\r\n```\r\nSince the bootstrap icon-set used in our themes produces an css-selector conflict in combination with this packages icon-set we couldn't stick with the standard. For an easy re-use it is seen as a better solution to extend the standard in an minimal way and keep the **icon** class as a clear icon-identifier. The font-specific appendix should be shorter then proposed but is open for discussion. See old vs. new:\r\n```html\r\n<i class=\"icon icon_re-property-management\">current state</i>\r\n<i class=\"psf psf-realestate-property-management\">new proposed</i>\r\n``` \r\nI don't see more meaning in the new proposal and several drawbacks, so I disagree with it.\r\n"}}