Agenda for 2017 February 7

The Theme Review Team holds a meeting weekly and we encourage all members to attend.

Channel: #themereview | Time: Tuesday at 18:00 UTC 18:00 UTC

Topics:

Social Media

There have been a few questions about integrating social media elements in a theme.

As theme features:

  1. Social media links with icons settings are allowed.
  2. Social media buttons to share posts is plugin territory.
  3. Social media “like” and “follow” buttons are plugin territory.

In the customizer with the theme authors links.

  1. Social media links are allowed.
  2. Social media “like” and “follow” buttons is not allowed.

REST API Theme

We have our first theme that is built in JS and used the REST API called “Foxhound”. @greenshady and @grapplerulrich reviewed it before it was uploaded to Trac. It could not be uploaded directly as would be blocked by the Theme Check plugin. We wanted to check the theme before we made an exception for it. After completing the review process we allowed it to bypass the Theme Check and be uploaded to trac: #39917 where it is now waiting for it’s accessibility review. Once it is passes the accessibility review we propose to set it live.

There is no need to keep it in the queue for 5 months. If we set it live in the near future or in 5 months it will not have any affect on when other themes will go live.

WordPress Stackoverflow Community Promotion Ads

WordPress Stackoverflow is looking for new Community Promotion Ads. @grapplerulrich thought it would be cool if we could add one for the theme review team. What we need is

  • an image
  • a tagline to show on mouseover

The image requirements are:

  • The image that you create must be 300 x 250 pixels, or double that if high DPI.
  • Must be GIF or PNG
  • If the background of the image is white or partially white, there must be a 1px border (2px if high DPI) surrounding it.

What do you think of the idea? Have we got some ideas for the image and tagline then add them in the comments below.

If you have any topics, then please reply to this post and include the topic, along with a brief description of what you would like to discuss.

Agenda for 2017 January 31

The Theme Review Team holds a meeting weekly and we encourage all members to attend.

Channel: #themereview | Time: Tuesday at 18:00 UTC 18:00 UTC

Topics:

Demo & starter content

In December we discussed what restrictions we wanted to apply to demo and starter content but in the last meeting we discovered that some of the ideas could be improved on. @poena has put a doc together to help list all of the pain points that we would like to improve. https://docs.google.com/document/d/1m8h9cTMlAq5xsDfWvvclsI54Pk_k0FB6kQVQbgIMZoA/edit?usp=sharing

Priority queue

Priority queue for theme authors who have had less than 5 issues in their last theme review and they would be removed if they have more than 5 issues in a review. The number of theme authors on the list may be small in the beginning but this will be a goal for theme authors to work to and provide additional motivation to make sure the themes are up to standard.

A meta ticket #meta-2456 has been created. What we need now is a list of theme authors who have had less than 5 issues in their last theme review. Please post the candidates in the comments with a link to the last theme ticket.

Example: User: greenshady Ticket: https://themes.trac.wordpress.org/ticket/35636

iframe

We are working on a Sniff to check for iframes. iframes can be a security risk as they can include content from other sites which can be malicious. The question that to be put up for discussion is: Do we want to create a whitelist of URLs that people are allowed to use for iframes? iframes can be used for embedding videos either as demo content or documentation. Are these valid use cases? Are there any other valid use cases?

The GitHub issue with a list of proposed URLs to whitelist: WPTRT/WordPress-Coding-Standards/#109

Improving theme author experience

In the last meeting we discussed the ideas that @ilovewpcom presented.

  • The adminbar with a login link has been enabled for wordpress.org/themes which is great!
  • For the two other issues Meta tickets have been created #meta-2445 and #meta-2444. Now we need people to help out with the implementation.

We should continue to work trying to find ways we can improve the theme author experience.

Social Media

There have been a few questions about integrating social media elements in a theme.

As theme features:

  1. Social media links with icons settings are allowed.
  2. Social media buttons to share posts is plugin territory.
  3. Social media “like” and “follow” buttons are plugin territory.

In the customizer with the theme authors links.

  1. Social media links with icons settings are allowed.
  2. Social media “like” and “follow” buttons is not allowed.

Theme Review Shindig

The next shindig is the weekend of February 4. The livecast will focus on hooks and filters. The livecast will start on Saturday at 14:00 UTC 14:00 UTC.

If you have any topics, then please reply to this post and include the topic, along with a brief description of what you would like to discuss.

Agenda for 2017 January 24

The Theme Review Team holds a meeting weekly and we encourage all members to attend.

Channel: #themereview | Time: Tuesday at 18:00 UTC 18:00 UTC

Topics:

Demo & starter content

In December we discussed what restrictions we wanted to apply to demo and starter content.

  • We define placeholder content as “Anything that is shown that you cannot pull from the user’s install” (10 thumbsup)
  • There must be a capability check so that placeholder content is only shown to logged in users.
  • For starter content the content provided by WordPress core must be used with an exception of images. (9 thumbsup)

Improving theme author experience

In the last meeting we discussed improving the theme author experience. @ilovewpcom has come up with a few suggestions how we can do this.

  • The theme upload process can be improved to reduce the rate of abandonment. Additionally encouraging theme authors to continue to improve on their themes which would decrease the amount of issues spotted on the first review. Themes with less issues would lead to less subsequent reviews and quicker approval times.
  • On the/themes/ page there is no login link. On the Upload Theme page it doesn’t show  your current account. Related Meta Ticket #meta-1050
  • The third UI issue is that even after getting to your account, there is no easy way to see all your current tickets (with themes in the queue). You have to remember or dig through emails in order to get to Trac, and even then you can’t easily see your current tickets which is difficult and confusing. (@grapplerulrich has create a new report [26]  that will show all of the tickets for themes that the user has uploaded.)

@ilovewpcom has come up with simple suggestions that would have a positive effect on the whole authorship/reviewing process. There are 5 simple mock-ups.

  • The red boxes in the first 3 screenshots show what has been added to the currently existing design.
  • The last 2 screenshots are new pages (including a new “Tickets” tab).

PHP7 Support

I propose to help users switch easily to PHP7 for their websites, themes on the w.org repo should be PHP7 compatible. This would mean themes should not have any PHP notices when running in PHP7. This is not be something reviewers will actively needs to test. There will be an automated check with the new Theme Check plugin that can check for PHP7 support. In the meantime you can the PHP Compatibility Checker plugin. The Theme Preview Site is now running PHP7 also.

Theme Review Shindig

The next shindig is the weekend of February 4. The livecast will focus on hooks and filters.

If you have any topics, then please reply to this post and include the topic, along with a brief description of what you would like to discuss.

X-post: Theme Developer Handbook released

X-post from +make.wordpress.org/docs: Theme Developer Handbook released

January 17, 2017 meeting notes

We had a fairly productive meeting with loads of good ideas. Read the Slack transcript here.

Hangout Ideas

Ideas proposed for the next hangout:

  • Working with the customizer, possibly discussing how best to implement options such as slides.
  • Using core hooks.

Disappearing reviewers

The team voted to automatically remove a reviewer if they have not added a comment in the ticket for 2 days after being assigned as the reviewer on a ticket. The ticket would then go back to the top of the queue so that another reviewer could pick it up.

Improving queue

Docs: The discussion evolved into improving the handbook and making things much clearer for theme authors and new reviewers. One proposal is to make February “theme handbook review” month, in which we spend time working on and improving the handbook.

Mentoring: There are a number of people who would like to bring the mentorship program back (+9 in favor). However, the concern is having enough experienced reviewers with the time to mentor new reviewers. If you’re a theme reviewer who would like to mentor, please comment below so that we can have a general count of who might be interested.

Priority queue: +13 votes were registered in favor of the following priority queue idea:

2) Priority queue for theme authors who have had less than 5 issues in their last theme review and they would be removed if they have more than 5 issues in a review. The number of theme authors on the list may be small in the beginning but this will be a goal for theme authors to work to and provide additional motivation to make sure the themes are up to standard. This should be relatively easy to implement with a custom report and perhaps some customisation to the assign review button.

Improving theme author experience

@ilovewpcom is in charge of working on the “theme author experience” project (just came up with that name). If you’re interested in helping out with making the uploading/reviewing experience better (from a theme author perspective) or sharing ideas, ping him on Slack.