{"action":"created","issue":{"url":"https://api.github.com/repos/jenius/axis/issues/38","labels_url":"https://api.github.com/repos/jenius/axis/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/jenius/axis/issues/38/comments","events_url":"https://api.github.com/repos/jenius/axis/issues/38/events","html_url":"https://github.com/jenius/axis/issues/38","id":15149952,"number":38,"title":"Style suggestion: Don't name mixins after HTML elements","user":{"login":"declandewet","id":1918632,"avatar_url":"https://avatars.githubusercontent.com/u/1918632?v=3","gravatar_id":"","url":"https://api.github.com/users/declandewet","html_url":"https://github.com/declandewet","followers_url":"https://api.github.com/users/declandewet/followers","following_url":"https://api.github.com/users/declandewet/following{/other_user}","gists_url":"https://api.github.com/users/declandewet/gists{/gist_id}","starred_url":"https://api.github.com/users/declandewet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/declandewet/subscriptions","organizations_url":"https://api.github.com/users/declandewet/orgs","repos_url":"https://api.github.com/users/declandewet/repos","events_url":"https://api.github.com/users/declandewet/events{/privacy}","received_events_url":"https://api.github.com/users/declandewet/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2013-06-05T01:32:31Z","updated_at":"2016-01-24T01:12:35Z","closed_at":"2013-06-06T21:56:40Z","body":"I ran into this issue while adding baseline grid support:\r\n\r\n![picture](https://a248.e.akamai.net/camo.github.com/156b6dccc82fc48d43643b682804dcad36507021/687474703a2f2f6779617a6f2e636f6d2f36393339306565376239643436663532666334313837303331313263383163622e706e67)\r\n\r\nIt took me a very long time to figure out what was causing it, roughly 2 hours or so. But I eventually found out what was wrong. I had these lines (roughly):\r\n\r\n```\r\nnav ul\r\nnav ol\r\n   some: property\r\n```\r\n\r\nIt turns out that instead of setting `some: property`, it was actually calling your `nav()` mixin instead, and erroring out with a really confusing error message.\r\n\r\nChanging it to this fixed the problem:\r\n\r\n```\r\nnav\r\n  ul, ol\r\n    some: property\r\n```\r\n\r\nAnd hence, I put forth this suggestion: don't name mixins after elements. It causes conflicts (that can be avoided as above, but not everyone will be as determined as I was to find out how to fix it, and they'll just complain)\r\n\r\nSo, instead of `nav()`, have it be `navbar()` or `navigation()` or something along those lines."},"comment":{"url":"https://api.github.com/repos/jenius/axis/issues/comments/174240025","html_url":"https://github.com/jenius/axis/issues/38#issuecomment-174240025","issue_url":"https://api.github.com/repos/jenius/axis/issues/38","id":174240025,"user":{"login":"declandewet","id":1918632,"avatar_url":"https://avatars.githubusercontent.com/u/1918632?v=3","gravatar_id":"","url":"https://api.github.com/users/declandewet","html_url":"https://github.com/declandewet","followers_url":"https://api.github.com/users/declandewet/followers","following_url":"https://api.github.com/users/declandewet/following{/other_user}","gists_url":"https://api.github.com/users/declandewet/gists{/gist_id}","starred_url":"https://api.github.com/users/declandewet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/declandewet/subscriptions","organizations_url":"https://api.github.com/users/declandewet/orgs","repos_url":"https://api.github.com/users/declandewet/repos","events_url":"https://api.github.com/users/declandewet/events{/privacy}","received_events_url":"https://api.github.com/users/declandewet/received_events","type":"User","site_admin":false},"created_at":"2016-01-24T01:12:35Z","updated_at":"2016-01-24T01:12:35Z","body":"I can confirm that this is no longer an issue in Stylus - If memory serves me correctly, this behaviour was caused by a recursion bug in Stylus and was indirectly fixed by feature additions circa 2014. Those \"best practices\" are super duper outdated, mind you. "}}