GitHub PullRequestReviewCommentEvent at 2014-11-22 01:02:46
Repository: scally
Actor: kllevin
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/westfieldlabs/scally/pulls/comments/20752450","id":20752450,"diff_hunk":"@@ -1,15 +1,240 @@\n ﻿# Scally Layout\n \n-Layout exists to prevent you from writing CSS that handles the most common\n-layout patterns in user interface design, right down to the component level.\n-Good comparisons for Layout are permanent scaffolding and Lego.\n \n-The main part of Layout is a powerful fluid and nestable mobile first grid\n-system allowing you to set custom column widths at different viewports, making\n-responsive layouts a sinch. The other part of Layout is a generic container\n-that ensures all of your UI uses consistent widths.\n \n-Demo: <http://codepen.io/team/westfieldlabs/full/cjyHK/>.\n \n-**N.B.** Layout styles should be prefixed with `l-` so that they're easily\n-identifiable.\n\\ No newline at end of file\n+## Contents\n+\n+- [What is layout?](#what-is-layout)\n+- [Why have layout?](#why-have-layout)\n+- [How to use](#how-to-use)\n+  - [Responsive grid](#responsive-grid)\n+  - [Use in isolation](#use-in-isolation)\n+- [Namespace](#namespace)\n+- [Demo's](#demos)\n+\n+\n+\n+\n+## What is layout?\n+\n+Scally features two powerful layout modules:\n+\n+### [Container](_l-container.scss)\n+\n+Container wraps all major UI layout pieces ensuring UI is consistent mainly with widths. Container is responsive in that it will be a fluid width using `%` up until it hits it's a maximum width where it's then becomes center aligned.\n+\n+### [Grid](_l-grid.scss)\n+\n+Grid is a powerful fluid and nestable mobile first grid system allowing you to set custom column widths at different viewports, making responsive layouts a sinch.\n+\n+See [demo's](#demos).\n+\n+Also in the [utilities](../utilities/) section of Scally there is the [**Side by Side** utility](../utilities/_u-side-by-side.scss) which applies an extremely common UI layout pattern. *This will be moved to the layout section, [see](https://github.com/westfieldlabs/scally/issues/153)*.\n+\n+*From global page template layouts right down to component level layouts, the above layout modules should take care of most—if not all—the main layout patterns you'll find in a UI.*\n+\n+Think of layout as Lego&reg;.\n+\n+\n+\n+\n+## Why have layout?\n+\n+Just like [utilities](../utilities/), layout can form a wide variety of UI layout patterns meaning as CSS authors you don't have to keep writing the same styles over and over again, instead you can abstract those common styles into nice reusable layout modules, mentioned above.\n+\n+Again like utilities, layout modules are extremely powerful and are the real work horses of any sort of UI build especially large-scale UI builds, and here are some reasons why:\n+\n+- Your CSS will be a lot DRYer.\n+- You can make far-reaching changes to your UI by simply modifying a layout module only once.","path":"layout/README.md","position":60,"original_position":60,"commit_id":"eae02144a4d2d5b6aafdd80644c7ab03060d0b8f","original_commit_id":"eae02144a4d2d5b6aafdd80644c7ab03060d0b8f","user":{"login":"kllevin","id":1783480,"avatar_url":"https://avatars.githubusercontent.com/u/1783480?v=3","gravatar_id":"","url":"https://api.github.com/users/kllevin","html_url":"https://github.com/kllevin","followers_url":"https://api.github.com/users/kllevin/followers","following_url":"https://api.github.com/users/kllevin/following{/other_user}","gists_url":"https://api.github.com/users/kllevin/gists{/gist_id}","starred_url":"https://api.github.com/users/kllevin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kllevin/subscriptions","organizations_url":"https://api.github.com/users/kllevin/orgs","repos_url":"https://api.github.com/users/kllevin/repos","events_url":"https://api.github.com/users/kllevin/events{/privacy}","received_events_url":"https://api.github.com/users/kllevin/received_events","type":"User","site_admin":false},"body":"You can make far-reaching changes to your UI with simple modifications to a single layout module.","created_at":"2014-11-22T01:02:46Z","updated_at":"2014-11-22T01:02:46Z","html_url":"https://github.com/westfieldlabs/scally/pull/154#discussion_r20752450","pull_request_url":"https://api.github.com/repos/westfieldlabs/scally/pulls/154","_links":{"self":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/comments/20752450"},"html":{"href":"https://github.com/westfieldlabs/scally/pull/154#discussion_r20752450"},"pull_request":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/154"}}},"pull_request":{"url":"https://api.github.com/repos/westfieldlabs/scally/pulls/154","id":24820546,"html_url":"https://github.com/westfieldlabs/scally/pull/154","diff_url":"https://github.com/westfieldlabs/scally/pull/154.diff","patch_url":"https://github.com/westfieldlabs/scally/pull/154.patch","issue_url":"https://api.github.com/repos/westfieldlabs/scally/issues/154","number":154,"state":"open","locked":false,"title":"Update documentation","user":{"login":"chris-pearce","id":1342772,"avatar_url":"https://avatars.githubusercontent.com/u/1342772?v=3","gravatar_id":"","url":"https://api.github.com/users/chris-pearce","html_url":"https://github.com/chris-pearce","followers_url":"https://api.github.com/users/chris-pearce/followers","following_url":"https://api.github.com/users/chris-pearce/following{/other_user}","gists_url":"https://api.github.com/users/chris-pearce/gists{/gist_id}","starred_url":"https://api.github.com/users/chris-pearce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chris-pearce/subscriptions","organizations_url":"https://api.github.com/users/chris-pearce/orgs","repos_url":"https://api.github.com/users/chris-pearce/repos","events_url":"https://api.github.com/users/chris-pearce/events{/privacy}","received_events_url":"https://api.github.com/users/chris-pearce/received_events","type":"User","site_admin":false},"body":"","created_at":"2014-11-21T05:52:45Z","updated_at":"2014-11-22T01:02:46Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5d3533cc50c61fae75fb84adc125e76054a28b9e","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/westfieldlabs/scally/pulls/154/commits","review_comments_url":"https://api.github.com/repos/westfieldlabs/scally/pulls/154/comments","review_comment_url":"https://api.github.com/repos/westfieldlabs/scally/pulls/comments/{number}","comments_url":"https://api.github.com/repos/westfieldlabs/scally/issues/154/comments","statuses_url":"https://api.github.com/repos/westfieldlabs/scally/statuses/eae02144a4d2d5b6aafdd80644c7ab03060d0b8f","head":{"label":"chris-pearce:update-documentation","ref":"update-documentation","sha":"eae02144a4d2d5b6aafdd80644c7ab03060d0b8f","user":{"login":"chris-pearce","id":1342772,"avatar_url":"https://avatars.githubusercontent.com/u/1342772?v=3","gravatar_id":"","url":"https://api.github.com/users/chris-pearce","html_url":"https://github.com/chris-pearce","followers_url":"https://api.github.com/users/chris-pearce/followers","following_url":"https://api.github.com/users/chris-pearce/following{/other_user}","gists_url":"https://api.github.com/users/chris-pearce/gists{/gist_id}","starred_url":"https://api.github.com/users/chris-pearce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chris-pearce/subscriptions","organizations_url":"https://api.github.com/users/chris-pearce/orgs","repos_url":"https://api.github.com/users/chris-pearce/repos","events_url":"https://api.github.com/users/chris-pearce/events{/privacy}","received_events_url":"https://api.github.com/users/chris-pearce/received_events","type":"User","site_admin":false},"repo":{"id":22742799,"name":"scally","full_name":"chris-pearce/scally","owner":{"login":"chris-pearce","id":1342772,"avatar_url":"https://avatars.githubusercontent.com/u/1342772?v=3","gravatar_id":"","url":"https://api.github.com/users/chris-pearce","html_url":"https://github.com/chris-pearce","followers_url":"https://api.github.com/users/chris-pearce/followers","following_url":"https://api.github.com/users/chris-pearce/following{/other_user}","gists_url":"https://api.github.com/users/chris-pearce/gists{/gist_id}","starred_url":"https://api.github.com/users/chris-pearce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chris-pearce/subscriptions","organizations_url":"https://api.github.com/users/chris-pearce/orgs","repos_url":"https://api.github.com/users/chris-pearce/repos","events_url":"https://api.github.com/users/chris-pearce/events{/privacy}","received_events_url":"https://api.github.com/users/chris-pearce/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/chris-pearce/scally","description":"Scally CSS framework - has no styling","fork":true,"url":"https://api.github.com/repos/chris-pearce/scally","forks_url":"https://api.github.com/repos/chris-pearce/scally/forks","keys_url":"https://api.github.com/repos/chris-pearce/scally/keys{/key_id}","collaborators_url":"https://api.github.com/repos/chris-pearce/scally/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/chris-pearce/scally/teams","hooks_url":"https://api.github.com/repos/chris-pearce/scally/hooks","issue_events_url":"https://api.github.com/repos/chris-pearce/scally/issues/events{/number}","events_url":"https://api.github.com/repos/chris-pearce/scally/events","assignees_url":"https://api.github.com/repos/chris-pearce/scally/assignees{/user}","branches_url":"https://api.github.com/repos/chris-pearce/scally/branches{/branch}","tags_url":"https://api.github.com/repos/chris-pearce/scally/tags","blobs_url":"https://api.github.com/repos/chris-pearce/scally/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/chris-pearce/scally/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/chris-pearce/scally/git/refs{/sha}","trees_url":"https://api.github.com/repos/chris-pearce/scally/git/trees{/sha}","statuses_url":"https://api.github.com/repos/chris-pearce/scally/statuses/{sha}","languages_url":"https://api.github.com/repos/chris-pearce/scally/languages","stargazers_url":"https://api.github.com/repos/chris-pearce/scally/stargazers","contributors_url":"https://api.github.com/repos/chris-pearce/scally/contributors","subscribers_url":"https://api.github.com/repos/chris-pearce/scally/subscribers","subscription_url":"https://api.github.com/repos/chris-pearce/scally/subscription","commits_url":"https://api.github.com/repos/chris-pearce/scally/commits{/sha}","git_commits_url":"https://api.github.com/repos/chris-pearce/scally/git/commits{/sha}","comments_url":"https://api.github.com/repos/chris-pearce/scally/comments{/number}","issue_comment_url":"https://api.github.com/repos/chris-pearce/scally/issues/comments/{number}","contents_url":"https://api.github.com/repos/chris-pearce/scally/contents/{+path}","compare_url":"https://api.github.com/repos/chris-pearce/scally/compare/{base}...{head}","merges_url":"https://api.github.com/repos/chris-pearce/scally/merges","archive_url":"https://api.github.com/repos/chris-pearce/scally/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/chris-pearce/scally/downloads","issues_url":"https://api.github.com/repos/chris-pearce/scally/issues{/number}","pulls_url":"https://api.github.com/repos/chris-pearce/scally/pulls{/number}","milestones_url":"https://api.github.com/repos/chris-pearce/scally/milestones{/number}","notifications_url":"https://api.github.com/repos/chris-pearce/scally/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/chris-pearce/scally/labels{/name}","releases_url":"https://api.github.com/repos/chris-pearce/scally/releases{/id}","created_at":"2014-08-08T02:04:16Z","updated_at":"2014-06-27T10:26:30Z","pushed_at":"2014-11-21T23:24:51Z","git_url":"git://github.com/chris-pearce/scally.git","ssh_url":"git@github.com:chris-pearce/scally.git","clone_url":"https://github.com/chris-pearce/scally.git","svn_url":"https://github.com/chris-pearce/scally","homepage":"","size":810,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"westfieldlabs:master","ref":"master","sha":"16d2610f537d867cf2957e7f802ae198ac788c0a","user":{"login":"westfieldlabs","id":4454594,"avatar_url":"https://avatars.githubusercontent.com/u/4454594?v=3","gravatar_id":"","url":"https://api.github.com/users/westfieldlabs","html_url":"https://github.com/westfieldlabs","followers_url":"https://api.github.com/users/westfieldlabs/followers","following_url":"https://api.github.com/users/westfieldlabs/following{/other_user}","gists_url":"https://api.github.com/users/westfieldlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/westfieldlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/westfieldlabs/subscriptions","organizations_url":"https://api.github.com/users/westfieldlabs/orgs","repos_url":"https://api.github.com/users/westfieldlabs/repos","events_url":"https://api.github.com/users/westfieldlabs/events{/privacy}","received_events_url":"https://api.github.com/users/westfieldlabs/received_events","type":"Organization","site_admin":false},"repo":{"id":21245669,"name":"scally","full_name":"westfieldlabs/scally","owner":{"login":"westfieldlabs","id":4454594,"avatar_url":"https://avatars.githubusercontent.com/u/4454594?v=3","gravatar_id":"","url":"https://api.github.com/users/westfieldlabs","html_url":"https://github.com/westfieldlabs","followers_url":"https://api.github.com/users/westfieldlabs/followers","following_url":"https://api.github.com/users/westfieldlabs/following{/other_user}","gists_url":"https://api.github.com/users/westfieldlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/westfieldlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/westfieldlabs/subscriptions","organizations_url":"https://api.github.com/users/westfieldlabs/orgs","repos_url":"https://api.github.com/users/westfieldlabs/repos","events_url":"https://api.github.com/users/westfieldlabs/events{/privacy}","received_events_url":"https://api.github.com/users/westfieldlabs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/westfieldlabs/scally","description":"Scally CSS framework","fork":false,"url":"https://api.github.com/repos/westfieldlabs/scally","forks_url":"https://api.github.com/repos/westfieldlabs/scally/forks","keys_url":"https://api.github.com/repos/westfieldlabs/scally/keys{/key_id}","collaborators_url":"https://api.github.com/repos/westfieldlabs/scally/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/westfieldlabs/scally/teams","hooks_url":"https://api.github.com/repos/westfieldlabs/scally/hooks","issue_events_url":"https://api.github.com/repos/westfieldlabs/scally/issues/events{/number}","events_url":"https://api.github.com/repos/westfieldlabs/scally/events","assignees_url":"https://api.github.com/repos/westfieldlabs/scally/assignees{/user}","branches_url":"https://api.github.com/repos/westfieldlabs/scally/branches{/branch}","tags_url":"https://api.github.com/repos/westfieldlabs/scally/tags","blobs_url":"https://api.github.com/repos/westfieldlabs/scally/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/westfieldlabs/scally/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/westfieldlabs/scally/git/refs{/sha}","trees_url":"https://api.github.com/repos/westfieldlabs/scally/git/trees{/sha}","statuses_url":"https://api.github.com/repos/westfieldlabs/scally/statuses/{sha}","languages_url":"https://api.github.com/repos/westfieldlabs/scally/languages","stargazers_url":"https://api.github.com/repos/westfieldlabs/scally/stargazers","contributors_url":"https://api.github.com/repos/westfieldlabs/scally/contributors","subscribers_url":"https://api.github.com/repos/westfieldlabs/scally/subscribers","subscription_url":"https://api.github.com/repos/westfieldlabs/scally/subscription","commits_url":"https://api.github.com/repos/westfieldlabs/scally/commits{/sha}","git_commits_url":"https://api.github.com/repos/westfieldlabs/scally/git/commits{/sha}","comments_url":"https://api.github.com/repos/westfieldlabs/scally/comments{/number}","issue_comment_url":"https://api.github.com/repos/westfieldlabs/scally/issues/comments/{number}","contents_url":"https://api.github.com/repos/westfieldlabs/scally/contents/{+path}","compare_url":"https://api.github.com/repos/westfieldlabs/scally/compare/{base}...{head}","merges_url":"https://api.github.com/repos/westfieldlabs/scally/merges","archive_url":"https://api.github.com/repos/westfieldlabs/scally/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/westfieldlabs/scally/downloads","issues_url":"https://api.github.com/repos/westfieldlabs/scally/issues{/number}","pulls_url":"https://api.github.com/repos/westfieldlabs/scally/pulls{/number}","milestones_url":"https://api.github.com/repos/westfieldlabs/scally/milestones{/number}","notifications_url":"https://api.github.com/repos/westfieldlabs/scally/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/westfieldlabs/scally/labels{/name}","releases_url":"https://api.github.com/repos/westfieldlabs/scally/releases{/id}","created_at":"2014-06-26T15:33:48Z","updated_at":"2014-11-21T23:18:22Z","pushed_at":"2014-11-21T23:18:22Z","git_url":"git://github.com/westfieldlabs/scally.git","ssh_url":"git@github.com:westfieldlabs/scally.git","clone_url":"https://github.com/westfieldlabs/scally.git","svn_url":"https://github.com/westfieldlabs/scally","homepage":"","size":5565,"stargazers_count":2,"watchers_count":2,"language":"CSS","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":8,"mirror_url":null,"open_issues_count":22,"forks":8,"open_issues":22,"watchers":2,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/154"},"html":{"href":"https://github.com/westfieldlabs/scally/pull/154"},"issue":{"href":"https://api.github.com/repos/westfieldlabs/scally/issues/154"},"comments":{"href":"https://api.github.com/repos/westfieldlabs/scally/issues/154/comments"},"review_comments":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/154/comments"},"review_comment":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/westfieldlabs/scally/pulls/154/commits"},"statuses":{"href":"https://api.github.com/repos/westfieldlabs/scally/statuses/eae02144a4d2d5b6aafdd80644c7ab03060d0b8f"}}}}