Make WordPress Themes

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4557 closed theme (live)

THEME: Matala - 1.5.7

Reported by: matt Owned by: garinungkadol
Priority: Keywords: theme-matala
Cc: m@…, otto@…, nacin@…


Matala - 1.5.7

Matala is a colorful, textured theme that supports two widget areas, six post formats, and several customization options, including custom background, custom header and an optional random image gallery on single image pages.

Theme URL - http://wordpress.org/extend/themes/matala
Author URL - http://www.gnvpartners.com

SVN - http://themes.svn.wordpress.org/matala/1.5.7
ZIP - http://wordpress.org/extend/themes/download/matala.1.5.7.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/matala/1.2&new_path=/matala/1.5.7

All previous tickets for this theme: http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-matala&order=priority


Attachments (3)

wide-images-1.jpg (26.2 KB) - added by garinungkadol 7 years ago.
wide-images-2.jpg (41.0 KB) - added by garinungkadol 7 years ago.
lists.jpg (38.7 KB) - added by garinungkadol 7 years ago.

Download all attachments as: .zip

Change History (11)

#1 @Otto42
7 years ago

  • Cc otto@… added

#2 @nacin
7 years ago

  • Cc nacin@… added

#3 @Otto42
7 years ago

Note to reviewer: This is almost a complete rewrite of the theme by Automattic's theme team for wordpress.com, with only minor changes made from their version to satisfy repo inclusion.

More info: http://en.blog.wordpress.com/2011/07/12/new-theme-matala/

Because of the extent of the changes, I expect a diff review will be way harder than normal, so feel free to do a normal review instead. Any recommendations given will be passed along to them as well.

#4 @emiluzelac
7 years ago

I noticed few things that I would like to recommend for Matala.

  • if you go to /layout-test/3/ and compare the typography with /comment-test/ you will noticed that Headings and Unordered List (Nested) are not styled the same way. Also Ordered List too.
  • Images Test > Other Image Tests > Wide Image, Resized In Editor: Image is 900x598px, resized in editor to 329×222. - The image does not resize to 329x222.

This is in accordance with Theme Unit Test only.

#5 @garinungkadol
7 years ago

  • Owner set to garinungkadol
  • Status changed from new to assigned

7 years ago

#6 @garinungkadol
7 years ago

This is a complete review. Below are the theme review results. Anything marked FAIL needs to be addressed. Anything marked REQUIRED is needed to be fixed while anything marked RECOMMENDED is strongly advised. Other points may be raised purely for information.

Just a couple changes are needed - mostly oversights so I'll keep the ticket open until you've done so. Please fix them on the next update.


Theme Check Results: PASS

Debugger results: PASS

Test data results: PASS but fixes are needed
Single Post View

  • REQUIRED: Ordered and unordered lists inside comments must be properly styled. See lists.jpg

Visual issues / functionality / browser checks: PASS

Validation: PASS


Code Quality: PASS but fixes are needed

  • REQUIRED: theme_options_render_page must be prefixed with the theme slug or an appropriate variant.
  • REQUIRED: 'Proudly powered by WordPress' in footer.php must have a text domain

If there are any issues once resolved, please re-submit your Theme using the uploader tool on wordpress.org/extend/themes, and a new Trac ticket will be generated.

  • Re-submitted Themes are given review priority over new Themes.

You can use the following tools to recreate the issues raised above if any have been:

Themes should be reviewed using define('WP_DEBUG', true); in wp-config.php
You can also view the Theme Review page.

Theme developer questions, comments, and feedback are welcome and encouraged. Please use one of the following methods:

  • 1) Leave a comment on this ticket
  • 2) Email the theme-reviewers email list
  • 3) Use the #wordpress-themes channel on irc.freenode.net
Last edited 7 years ago by garinungkadol (previous) (diff)

#7 @garinungkadol
7 years ago

  • Keywords syncapproved added
  • Resolution set to approved
  • Status changed from assigned to closed

#8 @chipbennett
7 years ago

  • Keywords syncapproved removed
Note: See TracTickets for help on using tickets.