Zbiór porad i informacji dla osób chcących zajmować się przeglądem motywów kandydujących do pojawienia się na oficjalnym repozytorium WordPress.org. Przygotowany z myślą o uczestnikach Contributor Day
2. 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
3. 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
nazwa_uzytkownika_na_wordpress_org@chat.wordpress.org
6. Jak otrzymać motyw do przeglądu?
https://make.wordpress.org/themes/handbook/review/
7. 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/
8. Struktura recenzji
• Kwestie wymagające naprawy (Required)
• Kwestie warte poprawienia (Recommended)
• Kwestie dodatkowe (Notes)
9. W przeglądzie motywu nie chodzi tylko
o przejście procedury przeglądu przez
motyw ale też o dzielenie się wiedzą
10. 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.
12. Ś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ć
13. 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
17. Weryfikujemy niuanse
(Notes) ;-)
• Według listy: https://make.wordpress.org/themes/
handbook/review/recommended/#design
• oraz własnych upodobań i wiedzy
19. 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.
20. 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/
21. 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
22. 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].
23. 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.
24. 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
26. Up selling
• Autorzy powinni używać ekranu personalizacji
motywu do up sellingu
• Zakazany jest agresywny up selling
27. 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
28. 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
29. Popularne pliki CSS/JS nie
powinny być prefiksowane
https://github.com/grappler/wp-standard-handles/