Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Author pages #81

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

Author pages #81

wants to merge 28 commits into from

Conversation

cheryllium
Copy link
Contributor

Changes in this branch: Added author pages and updated links throughout the site to link to them. Also removed media links in blog author bios.

@harrislapiroff
Copy link
Member

I think we should add icons to the social media links. People are fairly used to seeing them, such that it actually feels weirder to me not to see them. Also if you just want to find someone's twitter page, scanning for a bird is a lot faster than scanning for the word twitter.

(If you want to have it not be duplicating info, you could even display usernames/URLs instead of just "Twitter/GitHub/Portfolio")

@nmorduch
Copy link
Member

nmorduch commented Jul 6, 2016

Oh displaying usernames/URLs is a fine idea. Also UGH i guess we can camelcase GitHub.

I'll make a separate issue for banner images

@cheryllium
Copy link
Contributor Author

@harrislapiroff You bring up good points. I am now in favor of @nmorduch 's suggestion of putting the icons and the text together, and I like the idea of displaying usernames/URLs!

@nmorduch
Copy link
Member

nmorduch commented Jul 7, 2016

The author pages migration doesn't seem to be reversible. Maybe something to do with @chigby?

nmorduch added 2 commits July 27, 2016 14:22
Some models do not yet have migrations for the author pages.
We should merge and deploy, _then_ change those fields.
@nmorduch
Copy link
Member

We're having a weird migrations situation, so let's merge and deploy this branch, then make a few model changes and then deploy that. (This is because the author pages migration is janky.)

@harrislapiroff
Copy link
Member

Design-wise I would like for these links to be black or gray and for the photo to be part of the link:
screenshot 2016-08-12 17 33 11 There's other design things to change about the actual author page, but none critical and they should probably wait for adding more fields.

Otherwise... the trouble with migrations we've been having on this branch is suspect, but it seems to have worked OK on my computer. So technically I think this branch is OK to merge, but we should make sure we do it at a time when we all have bios prepped and someone is prepared to revert the database if needed.

@nmorduch
Copy link
Member

Cool cool maybe early next week? And then I'll do the rest of the work on this <3

@nmorduch
Copy link
Member

Made the changes on the about page @harrislapiroff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants