Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
Contributor Day
Przegląd motywów
Co jest wymagane by
sprawdzać motywy?
• Konto na wordpress.org
• Możliwość testowania motywów (posiadanie
lokalnego środow...
Przyda się też Slack
• Konto na chat.wordpress.org
• Pomoc w zakresie przeglądu motywów można uzyskać
na kanale #themerevi...
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...
Warto śledzić
https://make.wordpress.org/themes/
Na tym blogu pojawiają się informacje o
zmianach wymagań, procedur oraz r...
Jak otrzymać motyw do przeglądu?
https://make.wordpress.org/themes/handbook/review/
Od tego momentu mamy 48 godzin na
odezwanie się w przypisanym wątku
The team voted to automatically remove a
reviewer if t...
Struktura recenzji
• Kwestie wymagające naprawy (Required)
• Kwestie warte poprawienia (Recommended)
• Kwestie dodatkowe (...
W przeglądzie motywu nie chodzi tylko 

o przejście procedury przeglądu przez
motyw ale też o dzielenie się wiedzą
Przegląd odbywa się w formie konwersacji pomiędzy recenzentem
a autorem motywu, która odbywa się w wątku.
W wypadku znalez...
Zaczynamy przegląd
Środowisko testowe
• Instalujemy MAMP, LAMP, XAMP, WAMP (zależnie od systemu
operacyjnego): https://make.wordpress.org/cor...
Weryfikujemy czy autor nie
narusza licencji GPL
• Czy nie sprzedaje na swojej stronie lub Theme
Forest motywów niezgodnie z...
Licencje zgodne z GPL
• https://make.wordpress.org/themes/handbook/
review/resources/#gpl-compatible-
image%c2%a0licenses
...
Sprawdzamy niezbędne
wymagania (Required)
• Testujemy motyw wtyczką Theme Check
• Testujemy według listy na: https://
make...
Sprawdzamy wymagania
dodatkowe (Recommended)
• Według listy: https://make.wordpress.org/themes/
handbook/review/recommende...
Weryfikujemy niuanse
(Notes) ;-)
• Według listy: https://make.wordpress.org/themes/
handbook/review/recommended/#design
• o...
Warto pamiętać zanim
zaczniesz przegląd motywu
Zamykanie przeglądu ze względu na błędy bezpieczeństwa i
prefiksowania
 
Źródło: https://make.wordpress.org/themes/2016/07...
Przykładowa odpowiedź w takiej sytuacji:
 
Reviewers are now allowed to close the ticket if 3 or more security or
prefixin...
Zamykanie przeglądu ze względu na liczne błędy
 
Źródło: https://make.wordpress.org/themes/2016/08/23/meeting-summary-2016...
Szablon odpowiedzi:
 
I am sorry but because the following five errors are present I am
closing this ticket.
• #issue1

• ...
Ograniczenie liczby tagów tematycznych do 3
 
Źródło: https://make.wordpress.org/themes/2016/09/06/
meeting-summary-2016-0...
There have been a few questions about integrating social
media elements in a theme.
As theme features:
Social media links ...
Wtyczka czy motyw?
https://make.wordpress.org/themes/handbook/
review/required/#core-functionality-and-features
themes won...
Up selling
• Autorzy powinni używać ekranu personalizacji
motywu do up sellingu
• Zakazany jest agresywny up selling
Typowe błędy cz. 1
https://make.wordpress.org/themes/2017/03/25/review-statistics
Funkcje, które autorzy motywów implement...
Typowe błędy cz. 2
https://make.wordpress.org/themes/2017/03/25/review-statistics
• Missing escaping or using the wrong fu...
Popularne pliki CSS/JS nie
powinny być prefiksowane
https://github.com/grappler/wp-standard-handles/
Przydatne zasoby
Struktura plików motywu
https://wphierarchy.com/
Trochę o prefiksach
http://themereview.co/prefix-all-the-things/
Trochę o typowych błędach
https://make.wordpress.org/themes/2016/04/16/most-common-admin-review-issues/
Tagi motywów
https://make.wordpress.org/themes/handbook/review/required/theme-tags/
Upcoming SlideShare
Loading in …5
×

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

3,116 views

Published on

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

Published in: Technology
  • Be the first to comment

  • Be the first to like this

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

  1. 1. Contributor Day Przegląd motywów
  2. 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. 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
  4. 4. 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.
  5. 5. 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.
  6. 6. Jak otrzymać motyw do przeglądu? https://make.wordpress.org/themes/handbook/review/
  7. 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. 8. Struktura recenzji • Kwestie wymagające naprawy (Required) • Kwestie warte poprawienia (Recommended) • Kwestie dodatkowe (Notes)
  9. 9. W przeglądzie motywu nie chodzi tylko 
 o przejście procedury przeglądu przez motyw ale też o dzielenie się wiedzą
  10. 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.
  11. 11. Zaczynamy przegląd
  12. 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. 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
  14. 14. Licencje zgodne z GPL • https://make.wordpress.org/themes/handbook/ review/resources/#gpl-compatible- image%c2%a0licenses • https://www.gnu.org/licenses/license-list.pl.html
  15. 15. Sprawdzamy niezbędne wymagania (Required) • Testujemy motyw wtyczką Theme Check • Testujemy według listy na: https:// make.wordpress.org/themes/handbook/review/ required/
  16. 16. Sprawdzamy wymagania dodatkowe (Recommended) • Według listy: https://make.wordpress.org/themes/ handbook/review/recommended/
  17. 17. Weryfikujemy niuanse (Notes) ;-) • Według listy: https://make.wordpress.org/themes/ handbook/review/recommended/#design • oraz własnych upodobań i wiedzy
  18. 18. Warto pamiętać zanim zaczniesz przegląd motywu
  19. 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. 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. 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. 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. 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. 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
  25. 25. 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
  26. 26. Up selling • Autorzy powinni używać ekranu personalizacji motywu do up sellingu • Zakazany jest agresywny up selling
  27. 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. 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. 29. Popularne pliki CSS/JS nie powinny być prefiksowane https://github.com/grappler/wp-standard-handles/
  30. 30. Przydatne zasoby
  31. 31. Struktura plików motywu https://wphierarchy.com/
  32. 32. Trochę o prefiksach http://themereview.co/prefix-all-the-things/
  33. 33. Trochę o typowych błędach https://make.wordpress.org/themes/2016/04/16/most-common-admin-review-issues/
  34. 34. Tagi motywów https://make.wordpress.org/themes/handbook/review/required/theme-tags/

×