WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#24863 closed defect (bug) (fixed)

Twenty Fourteen: add a Contributors page template to highlight authors

Reported by: lancewillett Owned by:
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Bundled Theme Keywords: has-patch needs-testing
Focuses: Cc:

Description

For example: http://www.aeonmagazine.com/contributors/

  • Page template should list all authors with at least 1 published post.
  • Link to authors archive.

Attachments (13)

Screenshot from 2013-08-07 10:14:27.png (103.6 KB) - added by Kuzmanov 3 years ago.
style.diff (695 bytes) - added by Kuzmanov 3 years ago.
authors-page.php (1.4 KB) - added by Kuzmanov 3 years ago.
Contributors page template
authors.diff (2.4 KB) - added by MikeHansenMe 3 years ago.
author function and style from previous patch
authors-page.2.php (600 bytes) - added by MikeHansenMe 3 years ago.
page template to call author function and keep content and comments
24863.diff (2.4 KB) - added by MikeHansenMe 3 years ago.
based on new design.
24863.2.diff (2.7 KB) - added by MikeHansenMe 3 years ago.
minor update to previous patch
authors-page.3.php (1.6 KB) - added by MikeHansenMe 3 years ago.
contributor page file with all code within the file
24863.3.diff (1.0 KB) - added by MikeHansenMe 3 years ago.
styles for the newest author page file
24863.4.diff (6.2 KB) - added by obenland 3 years ago.
24863.5.diff (5.2 KB) - added by iamtakashi 3 years ago.
Minor style adjustments to 24863.4.diff
24863.6.diff (6.7 KB) - added by iamtakashi 3 years ago.
Include the missing contributors page template
24863.7.diff (6.5 KB) - added by obenland 3 years ago.

Download all attachments as: .zip

Change History (47)

#1 @Frank Klein
3 years ago

  • Cc contact@… added

#2 @melchoyce
3 years ago

  • Cc melissachoyce@… added

#3 @DrewAPicture
3 years ago

  • Cc xoodrew@… added

#4 @iamtakashi
3 years ago

  • Cc takashi@… added

@Kuzmanov
3 years ago

@Kuzmanov
3 years ago

Contributors page template

#5 @MikeHansenMe
3 years ago

  • Cc mdhansen@… added

A few ideas I have for this page are to also show the users gravatar if they have one and to still allow the page content to be displayed either below or above the authors. I have a template I have been playing with but it still needs a little work before I post it. Hopefully today sometime.

@MikeHansenMe
3 years ago

author function and style from previous patch

@MikeHansenMe
3 years ago

page template to call author function and keep content and comments

#6 @jazzs3quence
3 years ago

  • Cc jazzs3quence added

#7 @MikeHansenMe
3 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

#8 @obenland
3 years ago

I'd like to wait with that until iamtakashi had a chance to come up with a design for the page.

#9 @MikeHansenMe
3 years ago

The patch could easily be updated to meet a new design. Let's wait to get some input from iamtakashi.

#10 @iamtakashi
3 years ago

I'll work on a mockup for the contributors page.

#11 @SergeyBiryukov
3 years ago

  • Version set to trunk

#12 @iamtakashi
3 years ago

I've worked on a mockup for the contributors page.

http://f.cl.ly/items/1C261Z3C3b3q2O3G2033/2014-contributors.jpg

I think it's the best to order contibutors on the page by their number of published articles. Plus, to make the ordering clear, it would be good to print the number of articles on the button that takes to the author archive page. I used the document icon from Genericons in the mockup but it could say "Read xx Articles" or "View xx Articles" or whatever we think most appropriate. What do you guys think?

#13 follow-up: @obenland
3 years ago

I like it! What made you go with a one-column layout instead of a two- or three-column layout for example?

#14 in reply to: ↑ 13 @iamtakashi
3 years ago

Replying to obenland:

I like it! What made you go with a one-column layout instead of a two- or three-column layout for example?

Although we can use the Masonry, I found the layout can be messy with various text lengthes of biographies in two- or three-column layout and the readability will definitely suffer, and also the ordering will be obscured with multi-column Masonry powered layout.

Another reason was that, for that page, I wanted to use a good size of avatars, not like the ones in comments because these are more important on the page.

In my opinion, this one-column layout is cleaner and safer especially when a site has only a few contributors. Even a site has few dozens of contributors, the page wouldn't look horrible because it's clean and airy.

I'm glad you like it!

#15 @jazzs3quence
3 years ago

I like it a lot, too, and I agree with the design decisions you made re: one column layout.

#16 @lancewillett
3 years ago

Looks great, Takashi. Let's code it up. :)

@MikeHansenMe
3 years ago

based on new design.

#17 @MikeHansenMe
3 years ago

24863.diff uses author-page.2.php. It still allows for content if there is any. It will likely need more tuning, it is missing the icon in the button also.

@MikeHansenMe
3 years ago

minor update to previous patch

#18 @karmatosed
3 years ago

  • Cc karmatosed@… added

#19 @lancewillett
3 years ago

Discussed today in office hours, next step is testing and patch code review.

#20 @iamtakashi
3 years ago

I'll do tuning CSS once we are happy with the implementation.

#21 @obenland
3 years ago

Mike, could you see where using get_users() gets you? You can set it to order by post count and it should give you all the user information we need.

I'm not sure we actually need the .contributor-info div, we should be fine without it. .contributor-button should be outside the bio, see it more like meta info to the bio content. We might also be able to do all that in a page template.

#22 @MikeHansenMe
3 years ago

yep, I will see what I can do. Then submit a new patch.

@MikeHansenMe
3 years ago

contributor page file with all code within the file

@MikeHansenMe
3 years ago

styles for the newest author page file

@obenland
3 years ago

#23 follow-up: @obenland
3 years ago

24863.4.diff is based on MikeHansenMe's work and iamtakashi's mockup.

Takashi, could you make sure I got the colors, font-sizes, and widths/margins right?

#24 @jcastaneda
3 years ago

  • Cc jomcastaneda@… added

@iamtakashi
3 years ago

Minor style adjustments to 24863.4.diff

#25 in reply to: ↑ 23 ; follow-up: @iamtakashi
3 years ago

Replying to obenland:

Takashi, could you make sure I got the colors, font-sizes, and widths/margins right?

Adjusted slightly some styles. We can improve its responsive pattern when the patch in #25332 is committed.

#26 in reply to: ↑ 25 ; follow-up: @obenland
3 years ago

Replying to iamtakashi:

Adjusted slightly some styles. We can improve its responsive pattern when the patch in #25332 is committed.

Your patch is missing the page template :(

@iamtakashi
3 years ago

Include the missing contributors page template

#27 in reply to: ↑ 26 @iamtakashi
3 years ago

Replying to obenland:

Your patch is missing the page template :(

Oops. Sorry about that! Added a patch with the page template.

#28 follow-up: @lancewillett
3 years ago

Patch is looking great. One thing I noticed, and can be fixed in a later patch, is the word "Articles" isn't set up for translating numbers like 0, 1, n in various languages.

<?php printf( __( '%d Articles', 'twentyfourteen' ), $post_count ); ?>

and when an author has only 1 post, it still says "0 Articles" plural.

#29 in reply to: ↑ 28 @obenland
3 years ago

Replying to lancewillett:

Patch is looking great. One thing I noticed, and can be fixed in a later patch, is the word "Articles" isn't set up for translating numbers like 0, 1, n in various languages.

Good catch, let me update real quick.

#30 @lancewillett
3 years ago

Noting also the Genericons class should probably be removed and use CSS instead, as per changes in #25085. For adding the "document" icon.

#31 @obenland
3 years ago

Should I wait with a refresh until #25085 is in?

@obenland
3 years ago

#32 @lancewillett
3 years ago

In 25510:

Twenty Fourteen: first pass at adding a Contributors page template to highlight authors. Props MikeHansenMe, iamtakashi, obenland, and Kuzmanov. See #24863.

#33 @lancewillett
3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Let's open new issues in new tickets.

#34 @lancewillett
3 years ago

In 26657:

Twenty Fourteen: change sort order on Contributor template to put author with mosts posts at the top. Props iamtakashi, closes #26427; see #24863.

Note: See TracTickets for help on using tickets.