Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

restyle connected accounts #2124

Closed
wants to merge 1 commit into from
Closed

restyle connected accounts #2124

wants to merge 1 commit into from

Conversation

chadwhitacre
Copy link
Contributor

Here's an attempt at factoring out the Connected Accounts restyling from #2053. This is mostly done from a visual perspective, but the non-accounts-elsewhere Connected Accounts need to finish being wired up. I ran out of time.

@chadwhitacre chadwhitacre mentioned this pull request Mar 7, 2014
Closed
@seanlinsley
Copy link
Contributor

Are you going to continue working on this, or should someone else?

@seanlinsley
Copy link
Contributor

Translated: give me something to work on 🐱

@chadwhitacre
Get started on styling Connect Accounts
5653e03 
I made this commit by applying #2053.diff to the #2123 PR branch and
undoing any changes that weren't part of the Connected Accounts restyle.
It's mostly done from a visual perspective, though the non-accounts
elsewhere Connected Accounts need to finish being wired up. I ran out of
time.
@chadwhitacre
Copy link
Contributor Author

I'm closing this PR, because #2053 actually changes Connected Accounts quite a bit, and we need to break it down into even smaller PRs than this one. IRC

@chadwhitacre chadwhitacre deleted the restyle-connected-accounts branch March 13, 2014 21:22
@chadwhitacre chadwhitacre mentioned this pull request Mar 14, 2014
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chadwhitacre @seanlinsley