{"action":"created","issue":{"url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/4158","repository_url":"https://api.github.com/repos/SimpleMachines/SMF2.1","labels_url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/4158/labels{/name}","comments_url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/4158/comments","events_url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/4158/events","html_url":"https://github.com/SimpleMachines/SMF2.1/issues/4158","id":238737729,"number":4158,"title":"Dont format HTML outside of templates.","user":{"login":"NegativeIQ","id":2947012,"avatar_url":"https://avatars3.githubusercontent.com/u/2947012?v=3","gravatar_id":"","url":"https://api.github.com/users/NegativeIQ","html_url":"https://github.com/NegativeIQ","followers_url":"https://api.github.com/users/NegativeIQ/followers","following_url":"https://api.github.com/users/NegativeIQ/following{/other_user}","gists_url":"https://api.github.com/users/NegativeIQ/gists{/gist_id}","starred_url":"https://api.github.com/users/NegativeIQ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NegativeIQ/subscriptions","organizations_url":"https://api.github.com/users/NegativeIQ/orgs","repos_url":"https://api.github.com/users/NegativeIQ/repos","events_url":"https://api.github.com/users/NegativeIQ/events{/privacy}","received_events_url":"https://api.github.com/users/NegativeIQ/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-06-27T05:01:38Z","updated_at":"2017-07-08T17:37:50Z","closed_at":null,"body":"I noticed that for some reasons there is some HTML formatting done inside sources files.\r\nFor example, administration/moderation link has prepended <span class=\"amt\">%d</span> after button title. While that makes it easier for some users to make templates, it may limit (well at least annoy) others. I think that number of notifications should be as separate variable in array, and that there should not be any html in those variables at all. Also as far as i remember there is some similar things used in board and messages indexes. "},"comment":{"url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/comments/313870163","html_url":"https://github.com/SimpleMachines/SMF2.1/issues/4158#issuecomment-313870163","issue_url":"https://api.github.com/repos/SimpleMachines/SMF2.1/issues/4158","id":313870163,"user":{"login":"NegativeIQ","id":2947012,"avatar_url":"https://avatars3.githubusercontent.com/u/2947012?v=3","gravatar_id":"","url":"https://api.github.com/users/NegativeIQ","html_url":"https://github.com/NegativeIQ","followers_url":"https://api.github.com/users/NegativeIQ/followers","following_url":"https://api.github.com/users/NegativeIQ/following{/other_user}","gists_url":"https://api.github.com/users/NegativeIQ/gists{/gist_id}","starred_url":"https://api.github.com/users/NegativeIQ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NegativeIQ/subscriptions","organizations_url":"https://api.github.com/users/NegativeIQ/orgs","repos_url":"https://api.github.com/users/NegativeIQ/repos","events_url":"https://api.github.com/users/NegativeIQ/events{/privacy}","received_events_url":"https://api.github.com/users/NegativeIQ/received_events","type":"User","site_admin":false},"created_at":"2017-07-08T17:37:50Z","updated_at":"2017-07-08T17:37:50Z","body":"Its not the span element itself that's actual \"problem\". If we dont want to show it we can just add display: none, but that is actually a workaround. Also if we wanted to show that same notifications number without text elsewhere (in modal or popup perhaps), then we would have to cut it out using PHP inside templates. Its not some issue that doesnt have some easy workaround, but the fact we need a workaround for such thing shows flaws in software design. If you still feel like its not worth doing feel free to close issue."}}
{"actor":{"display_login":"NegativeIQ"}}