WordPress.org

Make WordPress Themes

#30949 closed theme (not-approved)

THEME: Cursos – 1.0.2

Reported by: stefan002377 Owned by: miiitaka
Priority: new theme Keywords: theme-cursos
Cc: stefan002377@…

Description

Cursos - 1.0.0

Cursos is a clean, modern and responsive WordPress theme, designed for courses wordpress theme, but It’s a multipurpose WordPress theme which can be used for a business site, a personal blog, portfolios and more. This theme is very easy to use for non coders also.

Theme URL - http://www.pwtthemes.com/theme/cursos-free-responsive-wordpress-theme
Author URL - http://www.stefanciobanu.com

SVN - https://themes.svn.wordpress.org/cursos/1.0.0
ZIP - https://wordpress.org/themes/download/cursos.1.0.0.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#30949 THEME: Cursos – 1.0.2 closed not-approved miiitaka

(this ticket)

#39187 THEME: Cursos – 1.0.3 closed not-approved acosmin
#46728 THEME: Cursos – 1.0.5 closed closed-newer-version-uploaded canyonthemes
#49150 THEME: Cursos – 1.0.6 reviewing poena


https://themes.svn.wordpress.org/cursos/1.0.0/screenshot.png

Change History (16)

#1 @themetracbot
17 months ago

  • Summary changed from THEME: Cursos – 1.0.0 to THEME: Cursos – 1.0.1

Cursos - 1.0.1

Cursos is a clean, modern and responsive WordPress theme, designed for courses wordpress theme, but It’s a multipurpose WordPress theme which can be used for a business site, a personal blog, portfolios and more. This theme is very easy to use for non coders also.

Theme URL - https://www.pwtthemes.com/theme/cursos-free-responsive-wordpress-theme
Author URL - http://www.stefanciobanu.com

SVN - https://themes.svn.wordpress.org/cursos/1.0.1
ZIP - https://wordpress.org/themes/download/cursos.1.0.1.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=cursos/1.0.0&new_path=cursos/1.0.1

History:

Ticket Summary Status Resolution Owner
#30949 THEME: Cursos – 1.0.2 closed not-approved miiitaka

(this ticket)

#39187 THEME: Cursos – 1.0.3 closed not-approved acosmin
#46728 THEME: Cursos – 1.0.5 closed closed-newer-version-uploaded canyonthemes
#49150 THEME: Cursos – 1.0.6 reviewing poena


https://themes.svn.wordpress.org/cursos/1.0.1/screenshot.png

#2 @themetracbot
16 months ago

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

#3 @danieltj
16 months ago

This will be my first theme to review, let's go! :)

#4 @grapplerulrich
16 months ago

  • Owner danieltj deleted

I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.

If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview or you can take on another review when you have time again.

#5 @themetracbot
16 months ago

  • Owner set to codepixelzmedia

#6 @codepixelzmedia
16 months ago

Hi Stefan, I shall review your theme and get back to you.

#7 @codepixelzmedia
16 months ago

Hi, I will get back in few minutes.

#8 @codepixelzmedia
16 months ago

Hi @stefan002377 , First of all, the theme looks nice :)
I went through it and here is my review:

  • Enqueued JS/CSS files should have your text-domain as prefix (functions.php)
  • All functions need to be pre-fixed with your text domain. (Eg: Line 21 of about.php)

Apart from that, I think we are good. Please upload updated theme with changes, and I shall go through them and we can mark this as approved.

#9 @grapplerulrich
15 months ago

@stefan002377 Please upload an update or the ticket will be closed.

  • Please correct the default copyright text "Copyright 2016 Cursos Theme All Rights Reserved." The sites copyright does not belong to your theme.
  • There is no need to escape hardcoded data e.g. esc_url('http://wordpress.org/') as I can that the data is secure.

Enqueued JS/CSS files should have your text-domain as prefix

This is not necessary as they are third party scripts and by keeping the same handle it helps prevent the same script being loaded twice.

#10 @themetracbot
15 months ago

  • Summary changed from THEME: Cursos – 1.0.1 to THEME: Cursos – 1.0.2

Cursos - 1.0.2

Cursos is a clean, modern and responsive WordPress theme, designed for courses wordpress theme, but It’s a multipurpose WordPress theme which can be used for a business site, a personal blog, portfolios and more. This theme is very easy to use for non coders also.

Theme URL - https://www.pwtthemes.com/theme/cursos-free-responsive-wordpress-theme
Author URL - http://www.stefanciobanu.com

SVN - https://themes.svn.wordpress.org/cursos/1.0.2
ZIP - https://wordpress.org/themes/download/cursos.1.0.2.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=cursos/1.0.1&new_path=cursos/1.0.2

History:

Ticket Summary Status Resolution Owner
#30949 THEME: Cursos – 1.0.2 closed not-approved miiitaka

(this ticket)

#39187 THEME: Cursos – 1.0.3 closed not-approved acosmin
#46728 THEME: Cursos – 1.0.5 closed closed-newer-version-uploaded canyonthemes
#49150 THEME: Cursos – 1.0.6 reviewing poena


https://themes.svn.wordpress.org/cursos/1.0.2/screenshot.png

#11 @grapplerulrich
15 months ago

  • Owner codepixelzmedia deleted

I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.

If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview or you can take on another review when you have time again.

#12 @themetracbot
15 months ago

  • Owner set to miiitaka

#13 @miiitaka
15 months ago

Hi @stefan002377,

I began reviewing the theme.
Give me around 5 days until I post my first review here.

The whole review process will be as described below.

  1. I review the theme according to theme review handbook and post the result here as a comment.
  2. You solve all the issues.
  3. You go to https://wordpress.org/themes/upload/ and reupload the new version.
    • The theme must have the same name
    • You need to change the Version tag in style.css
    • New version will be attached to this ticket.
    • It's better to log your changes in document.
  4. We repeat going through the process no.1 to 3 until I find no issues any more.
  5. I approve the theme and this ticket will have a status approved.
  6. One of the admin will check this theme and review the theme himself/herself and make it live. It is possible that the admin reviewer adds more issues. I'm a new reviewer and need double check.

#14 @miiitaka
15 months ago

Hi @stefan002377,

Today is review the contents.

required

(Required are something you need to fix to get approved.)

  • none.

(Recommended are something you don't need to fix to get approved.)

  • Screenshot size should be 1200x900, to account for HiDPI displays. Any 4:3 image size is acceptable, but 1200x900 is preferred.
  • No reference to add_theme_support( "custom-header", $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
  • Typo: Comment text "section" is set to "sections".
  • Is set in the style sheet "hgroup" element is better you do not set because it is deprecated.
  • Empty folder "lang" exists. Let's remove so unnecessary.

Notes:

(Notes are something I want to talk with you but what you'll do with these issues is up to you and you don't need to answer.)

  • At least one hard coded date was found in the file single.php. Consider get_option( 'date_format' ) instead.
  • At least one hard coded date was found in the file content-posts.php. Consider get_option( 'date_format' ) instead.
  • front-page.php The theme appears to use include or require. If these are being used to include separate sections of a template from independent files, then get_template_part() should be used instead.

Line 9: include( get_home_template() );

Thanks! :)

#15 @grapplerulrich
15 months ago

@stefan002377 If there is no update the ticket will be closed.

#16 @grapplerulrich
15 months ago

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

Closing the ticket due to no response.

Note: See TracTickets for help on using tickets.