WordPress.org

Make WordPress Themes

Opened 5 years ago

Closed 5 years ago

#12632 closed theme (not-approved)

THEME: Metro Style Marketing - 1.0.3

Reported by: semdashboard Owned by: aymanalzarrad
Priority: previously reviewed Keywords: theme-metro-style-marketing
Cc: info@…

Description

Metro Style Marketing - 1.0.3

Metro Style Marketing is a theme made with the Internet and Affiliate marketers in mind. Its layout is the ideal solution to get the best conversion rate from the visitors. In the first section, the webmaster can insert a big image of the services or the products provided. The opt-in form grabs the attention of the visitor and stimulate them to digit their email address. Metro Style Marketing is a simple, clean and responsive WordPress Theme which adapts automatically to your tablets and mobile devices. Along with the metro style design the theme is easily customizable with numerous theme options. It is translation ready. Get free support on http://www.semdashboard.com/

Theme URL - http://www.semdashboard.com/metro-style-marketing/
Author URL - http://www.semdashboard.com/

SVN - http://themes.svn.wordpress.org/metro-style-marketing/1.0.3
ZIP - http://wordpress.org/themes/download/metro-style-marketing.1.0.3.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/metro-style-marketing/1.0.2&new_path=/metro-style-marketing/1.0.3

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-metro-style-marketing&order=id

https://themes.svn.wordpress.org/metro-style-marketing/1.0.3/screenshot.png

Change History (6)

#1 @chipbennett
5 years ago

  • Owner set to stumpfmi
  • Status changed from new to reviewing

#2 follow-up: @chipbennett
5 years ago

@stumpfmi are you still able to review this ticket?

#3 in reply to: ↑ 2 @stumpfmi
5 years ago

Replying to chipbennett:

@stumpfmi are you still able to review this ticket?

I am sorry, things got very busy after I requested the ticket. It might be best to pass it on to someone else at the moment.

#4 @chipbennett
5 years ago

  • Owner changed from stumpfmi to aymanalzarrad

#5 @aymanalzarrad
5 years ago

Hi @semdashboard,
Some of the previously addressed issues were fixed but unfortunately I came across some more:

Code:

Styles and scripts shouldn't be hard coded and should always be included using: wp_enqueue_style() and wp_enqueue_script() ( You can find styles hard coded in functions.php on line 774 ).

Errors and notices:

  1. On Gallery post format page the theme is generating a warning: ( The error is generated and found in functions.php on line 314 )

WARNING: wp-includes/plugin.php:173 - call_user_func_array() expects parameter 1 to be a valid callback, function 'msm_image_navigation' not found or invalid function name

  1. When the widget "The Bootstrap Image Meta Widget" is not in use and you visite a single image gallery page the theme is generating a Notice:

NOTICE: wp-includes/widgets.php:1124 - Undefined index: The_Bootstrap_Image_Meta_Widget

Functionality:

  1. Password protected posts/page on index/archive page are not accepting the submit button for the password form.
  2. Password protected posts/page on sinlge page are leading to 404 error page after submitting the password

Layout:

  1. Extremely big images with caption are not being cropped properly and that is causing the rest of the image and caption to be hidden behind the next element/sidebar.
  2. Menu with many elements/pages is over edging and covering the elements/Boxes that are below the menu.
  3. Menu with high number of "child of child" level of depth elements/pages are over edging the screen width.

Suggestions:

I suggest you consider the following:

  1. Check the footer.php for the link to wordpress.org becasue the a tag has extra spaces which is causing the link to not work properly.
  2. The Boxes on home page are not handling well the non breaking text, for which you can use the following CSS properties to help you support non breaking text:
    -ms-word-wrap: break-word;
    word-wrap: break-word;
    
  1. Header 2 "h2" is bigger in size than the "h1" which is usually the opposite and I think that It could be confusing for the end-user.
  2. The HTML validation check returned several errors that you should take in consideration.

Final note:

Nice theme that you have here but please make sure to check your theme before re-uploading to avoid any further "not-approved" following the requirements and suggestions here: http://codex.wordpress.org/Theme_Review

If you have any questions please feel free to ask using the comments.

Last edited 5 years ago by aymanalzarrad (previous) (diff)

#6 @aymanalzarrad
5 years ago

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

I didn't hear back from you about the addressed issues, so this ticket is now closed.

Note: See TracTickets for help on using tickets.