Make WordPress Themes

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#78127 closed theme (not-approved)

THEME: Samaan – 1.1.3

Reported by: bohuan's profile bohuan Owned by: ravisakya's profile ravisakya
Priority: previously reviewed Keywords: theme-samaan child-theme parent-twentynineteen
Cc: bohuan92@…

Change History (3)

#1 @ravisakya
6 years ago

  • Keywords changed from theme-samaan, child-theme, parent-twentynineteen to theme-samaan child-theme parent-twentynineteen
  • Owner set to ravisakya
  • Status changed from new to reviewing

#2 @ravisakya
6 years ago

  • Resolution set to not-approved
  • Status changed from reviewing to closed

@bohuan

Themes that have 3 or more distinct issues may be closed as not-approved. However, theme authors may resubmit the theme once they’ve corrected the issues.

Issues

1. Accessibility

Keyboard Navigation:- In mobile the design is all over the place. I cannot test the keyboard navigation.

https://i.gyazo.com/9786779463f42e6715e973daf9c136b3.png

2. Code

There should not be any php or js errors.

https://i.gyazo.com/05ddb4880d4634a8290cd6dd500e5116.png

All untrusted data should be escaped before output.
https://themes.trac.wordpress.org/browser/samaan/1.1.3/comments.php#L40 get_the_title needs escaping

Install Theme Sniffer plugin and fix all the issues https://wordpress.org/plugins/theme-sniffer/

3. Selling, credits, and links

REQUIRED: Using a WordPress.org Theme URI is reserved for official themes.

Theme URI is optional
If used, it must be about the theme we’re hosting on WordPress.org

See here https://make.wordpress.org/themes/handbook/review/required/#selling-credits-and-links

4. Screenshot

The screenshot should be no bigger than 1200 x 900px
The ratio of width to height should be 4:3

ERROR The size of your screenshot should not exceed 1200x900, but screenshot.png is currently 1583x1992.

5. Licensing

Please add licensing in your readme.txt for all third party code (Bootstrap, Prettyphoto etc.)

This ticket was mentioned in Slack in #themereview by cyclonetheme. View the logs.


6 years ago

Note: See TracTickets for help on using tickets.