34
Contributor Day Przegląd motywów

Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Embed Size (px)

Citation preview

Page 1: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Contributor DayPrzegląd motywów

Page 2: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Co jest wymagane by sprawdzać motywy?

• Konto na wordpress.org

• Możliwość testowania motywów (posiadanie lokalnego środowiska do testów)

• Wiedza o strukturze motywów w WordPressie

• Podstawowa wiedza o HTML, CSS, JavaScript i PHP

Page 3: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Przyda się też Slack

• Konto na chat.wordpress.org

• Pomoc w zakresie przeglądu motywów można uzyskać na kanale #themereview

• Opis jak założyć konto: https://make.wordpress.org/chat/ Uwaga! Logowanie odbywa się poprzez mail postaci [email protected]

Page 4: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Trac

https://themes.trac.wordpress.org/

Dostęp do wątków uzyskuje się przez konto WordPress.org - dlatego należy je założyć przed rozpoczęciem prac przy przeglądaniu motywów.

Page 5: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Warto śledzić

https://make.wordpress.org/themes/

Na tym blogu pojawiają się informacje o zmianach wymagań, procedur oraz raporty ze

spotkań zespołu ds. przeglądu motywów.

Page 6: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Jak otrzymać motyw do przeglądu?

https://make.wordpress.org/themes/handbook/review/

Page 7: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Od tego momentu mamy 48 godzin na odezwanie się w przypisanym wątku

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

Źródło: https://make.wordpress.org/themes/2017/01/17/january-17-2017-meeting-notes/

Page 8: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Struktura recenzji

• Kwestie wymagające naprawy (Required)

• Kwestie warte poprawienia (Recommended)

• Kwestie dodatkowe (Notes)

Page 9: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

W przeglądzie motywu nie chodzi tylko o przejście procedury przeglądu przez motyw ale też o dzielenie się wiedzą

Page 10: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Przegląd odbywa się w formie konwersacji pomiędzy recenzentem a autorem motywu, która odbywa się w wątku.

W wypadku znalezienia problemów z motywem, autor ma 7 dni na odpowiedź, w przeciwnym wypadku wątek zostanie zamknięty z powodu braku aktywności. Tak samo recenzent ma 7 dni na odpowiedź odnośnie poprawek - w wypadku gdy brakuje nam na to czasu, warto podtrzymać konwersację aby wątek nie został zamknięty lub nam zabrany.

Page 11: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Zaczynamy przegląd

Page 12: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Środowisko testowe• Instalujemy MAMP, LAMP, XAMP, WAMP (zależnie od systemu

operacyjnego): https://make.wordpress.org/core/handbook/tutorials/installing-a-local-server/

• Instalujemy WordPressa

• Instalujemy wtyczki Theme Check, Monster Widget, Log Deprecated Notices, Query Monitor, Regenerate Thumbnails

• Ustawiamy zmienną WP_DEBUG na TRUE w wp-config.php

• Instalujemy dane testowe: https://codex.wordpress.org/Theme_Unit_Test

• W tym miejscu warto zrobić backup WordPressa :)

• Instalujemy motyw, który będziemy recenzować

Page 13: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Weryfikujemy czy autor nie narusza licencji GPL

• Czy nie sprzedaje na swojej stronie lub Theme Forest motywów niezgodnie z licencją GPL (głównie limit stron na których można używać motywu dyskwalifikuje)

• Sprawdzamy czy motyw używa zdjęć, grafik, fontów, bibliotek zgodnych z licencją GPL

• Upewniamy się, że miniaturka motywu używa (o ile w ogóle używa) wyłącznie zdjęć na licencji zgodnej z GPL

Page 15: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Sprawdzamy niezbędne wymagania (Required)

• Testujemy motyw wtyczką Theme Check

• Testujemy według listy na: https://make.wordpress.org/themes/handbook/review/required/

Page 16: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Sprawdzamy wymagania dodatkowe (Recommended)

• Według listy: https://make.wordpress.org/themes/handbook/review/recommended/

Page 17: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Weryfikujemy niuanse (Notes) ;-)

• Według listy: https://make.wordpress.org/themes/handbook/review/recommended/#design

• oraz własnych upodobań i wiedzy

Page 18: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Warto pamiętać zanim zaczniesz przegląd motywu

Page 19: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Zamykanie przeglądu ze względu na błędy bezpieczeństwa i prefiksowania Źródło: https://make.wordpress.org/themes/2016/07/12/meeting-notes-for-2016-july-12/  • Themes that do not fullfill the review baseline will be closed. The

ticket will be closed if the theme has more then 3 prefixing or security issues. This also applies to the admin review queue.

• It is not the end when a ticket is closed. The discussion can still continue and an update can always be updated.

Page 20: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Przykładowa odpowiedź w takiej sytuacji: Reviewers are now allowed to close the ticket if 3 or more security or prefixing issues are found.

This theme is being closed because the following security/prefixing issues have been found :

Please do not get discouraged by the ticket being closed. If you have questions you can ask them here or in the #themereview channel on Slack. Fix these issues and ensure you have checked your theme for additional security or prefixing problems.

You can read up more on security and prefixing on our resources page: https://make.wordpress.org/themes/handbook/review/resources/

Also take this opportunity to carefully review your theme against theme requirements listed in the reference below.

• ref: https://make.wordpress.org/themes/handbook/review/required/

Once the issues are fixed upload the new version and we will review it again.https://wordpress.org/themes/getting-started/

Page 21: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Zamykanie przeglądu ze względu na liczne błędy Źródło: https://make.wordpress.org/themes/2016/08/23/meeting-summary-2016-23-08/ W wypadku gdy motyw posiada 5 poważnych błędów, można odgórnie zakończyć przegląd - warto przy tym skorzystać z poniższego szablonu odpowiedzi. Przy czym błędy grupujemy według rodzaju: Closing tickets should be done for 5 or more distinct issues. Examples: • Theme has multiple places where text strings are not translatable – group these into 1

distinct issue

• Using custom functions instead of core functions such as the_posts_pagination, the_archive_title, the_excerpt – group into 1 issue

Page 22: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Szablon odpowiedzi: I am sorry but because the following five errors are present I am closing this ticket.

• #issue1

• #issue2

• #issue3

• #issue4

• #issue5

Do not get discouraged. A closed ticket is not the end. We want you to resubmit your theme. We do ask that you spend a bit more time making sure that your theme meets all [https://make.wordpress.org/themes/handbook/review/required/ the requirements] before we make it live.

If you need help with anything or want me to explain further, don’t hesitate to post here in this ticket. I’m happy to answer any questions you have. Or, feel free to post over on our [https://make.wordpress.org/chat/ Slack] [https://wordpress.slack.com/messages/themereview channel].

Page 23: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Ograniczenie liczby tagów tematycznych do 3 Źródło: https://make.wordpress.org/themes/2016/09/06/meeting-summary-2016-09-06/ Motyw może używać do 3 tagów dotyczących jego tematyki (blog, education itp.), pozostałe tagi nie mają limitu. Because we’re seeing more and more themes adding upwards of all 9 subject tags, we decided on a limit of 3 subject tags per theme. This goes into effect immediately.

Page 24: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

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

As theme features:

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

In the customizer with the theme authors links.

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

https://make.wordpress.org/themes/2017/01/29/agenda-for-2017-january-31/

Social media w motywach

Page 25: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Wtyczka czy motyw?

https://make.wordpress.org/themes/handbook/review/required/#core-functionality-and-features

themes won’t be allowed to add a “Custom CSS” box to their themes – falls under core functionality first requirement

Page 26: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Up selling

• Autorzy powinni używać ekranu personalizacji motywu do up sellingu

• Zakazany jest agresywny up selling

Page 27: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Typowe błędy cz. 1

https://make.wordpress.org/themes/2017/03/25/review-statistics

Funkcje, które autorzy motywów implementowali sami zamiast użyć API WordPressa:

• Logo • Custom CSS • Custom excerpt • Custom pagination

Page 28: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Typowe błędy cz. 2

https://make.wordpress.org/themes/2017/03/25/review-statistics

• Missing escaping or using the wrong functions • Text that is not translation ready • Missing prefix • Missing license or copyright information for included assets • Unused code or files • PHP notices, errors or warnings • Missing sanitization, or using the wrong functions • Options in the customizer that are not working • Scripts or styles are not enqueued • Style tags does not correspond with theme functionality, or are

deprecated

Page 29: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Popularne pliki CSS/JS nie powinny być prefiksowane

https://github.com/grappler/wp-standard-handles/

Page 30: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Przydatne zasoby

Page 31: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Struktura plików motywu

https://wphierarchy.com/

Page 32: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Trochę o prefiksach

http://themereview.co/prefix-all-the-things/

Page 33: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Trochę o typowych błędach

https://make.wordpress.org/themes/2016/04/16/most-common-admin-review-issues/

Page 34: Contributor Day - WordCamp Lublin 2017 - przegląd motywów

Tagi motywów

https://make.wordpress.org/themes/handbook/review/required/theme-tags/