WordPress.org

Make WordPress Themes

Opened 3 months ago

Closed 4 weeks ago

#46726 closed theme (live)

THEME: Rhymes – 1.5

Reported by: themescastle Owned by: kafleg
Priority: new theme Keywords: theme-rhymes
Cc: themes.castle99@…

Description

Rhymes - 1.0

Rhymes is simple blog theme. Rhymes fully supported Bootstrap mesonry layout. Rhymes theme provide unlimited color scheme.

Theme URL -
Author URL - http://darshansaroya.com

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.0

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

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.0/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.
  • RECOMMENDED: Tags: is either empty or missing in style.css header.

Change History (19)

#1 @poena
3 months ago

Hi!
This is not a complete review,
Please make sure that you add human readable, non minified versions of all minified files.
Reference: https://make.wordpress.org/themes/handbook/review/required/#stylesheets-and-scripts

Translated texts inside html attributes also needs to be escaped with esc_attr().

#2 @themetracbot
3 months ago

  • Summary changed from THEME: Rhymes – 1.0 to THEME: Rhymes – 1.1

Rhymes - 1.1

Rhymes is simple blog theme. Rhymes fully supported Bootstrap mesonry layout. Rhymes theme provide unlimited color scheme.

Theme URL -
Author URL - http://darshansaroya.com

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.1

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=rhymes/1.0&new_path=rhymes/1.1

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.1/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.
  • RECOMMENDED: Tags: is either empty or missing in style.css header.

#3 @themetracbot
3 months ago

  • Summary changed from THEME: Rhymes – 1.1 to THEME: Rhymes – 1.2

Rhymes - 1.2

Rhymes is simple blog theme. Rhymes fully supported Bootstrap mesonry layout. Rhymes theme provide unlimited color scheme.

Theme URL -
Author URL -

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.2

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=rhymes/1.1&new_path=rhymes/1.2

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.2/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.
  • RECOMMENDED: Author URI: is missing from your style.css header.

#4 @juhise
7 weeks ago

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

Hello @themescastle,

I am taking this theme to review, will post my feedback soon.

Best Regards,
Juhi

#5 @rabmalin
6 weeks ago

  • Owner juhise deleted

@juhise You can post review comments if you have already reviewed the theme. But theme needs to be returned to the queue. It could be officially taken for review when it is in top 5 on the review queue.

#6 @juhise
6 weeks ago

@rabmalin

Okay, Thanks for letting me know.

Is there any process got changed, because last time I think rule was "You can pick up any theme to review from first page".

Best Regards,
Juhi

#8 in reply to: ↑ 7 @juhise
6 weeks ago

#9 @kafleg
4 weeks ago

  • Owner set to kafleg

Hello @themescastle,

I am going to review this theme. If you want to update this theme update within 24 hours otherwise I will continue review on this. Reviewers are now allowed to close the ticket if the theme has 3+ distinct issues. https://make.wordpress.org/themes/2017/11/16/meeting-notes-for-2017-november-14/ Make sure your theme is developed based on the guidelines. https://make.wordpress.org/themes/handbook/review/required/
Before re-submitting themes, please check Theme Review Guidelines very thoroughly. Also, test your theme with following plugins.
https://wordpress.org/plugins/theme-check/
https://github.com/WPTRT/theme-sniffer/releases/download/0.1.4/ns-theme-check.0.1.4.zip [Errors need to be fixed and Warnings are things that need to be checked manually.]
Hope all the issues mentioned by the previous reviewer is fixed in this version.
Regards
@kafleg

#10 @themescastle
4 weeks ago

Ok, I will check theme with recommended plugins and update as soon.

#11 @themetracbot
4 weeks ago

  • Summary changed from THEME: Rhymes – 1.2 to THEME: Rhymes – 1.3

Rhymes - 1.3

Rhymes is simple multipupose blog theme. Rhymes fully supported Bootstrap masonry layout. Rhymes theme provide unlimited color scheme. Rhymes have post slider on front page, where you can manage posts by category. Rhymes theme is designed for blog, but one can use it for news or education site.

Theme URL -
Author URL - http://darshansaroya.com

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.3

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=rhymes/1.2&new_path=rhymes/1.3

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.3/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.
  • RECOMMENDED: Tags: is either empty or missing in style.css header.

#12 @themescastle
4 weeks ago

Hello @kafleg

I have tested the theme with recommended plugins and resolved all the issues.

Thanks

#13 @kafleg
4 weeks ago

Hello @themescastle,
Here are some issues you need to fix it before theme live.

Required

Thank you

#14 @themetracbot
4 weeks ago

  • Summary changed from THEME: Rhymes – 1.3 to THEME: Rhymes – 1.4

Rhymes - 1.4

Rhymes is simple multipupose blog theme. Rhymes fully supported Bootstrap masonry layout. Rhymes theme provide unlimited color scheme. Rhymes have post slider on front page, where you can manage posts by category. Rhymes theme is designed for blog, but one can use it for news or education site.

Theme URL -
Author URL - http://darshansaroya.com

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.4

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=rhymes/1.3&new_path=rhymes/1.4

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.4/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.
  • RECOMMENDED: Tags: is either empty or missing in style.css header.

#15 @themescastle
4 weeks ago

Hello @kafleg,

I have removed the double tag from style.css and also check for theme-slug functions and corrected where i got it.

Added new Screenshot of required size.

Hope this fulfil all requirements.

Thanks.

#16 follow-up: @kafleg
4 weeks ago

Check your latest screenshot, it's not good. The menu, site title appear in the content section also. Make it fine.

#17 @themetracbot
4 weeks ago

  • Summary changed from THEME: Rhymes – 1.4 to THEME: Rhymes – 1.5

Rhymes - 1.5

Rhymes is simple multipupose blog theme. Rhymes fully supported Bootstrap masonry layout. Rhymes theme provide unlimited color scheme. Rhymes have post slider on front page, where you can manage posts by category. Rhymes theme is designed for blog, but one can use it for news or education site.

Theme URL -
Author URL - http://darshansaroya.com

Trac Browser - https://themes.trac.wordpress.org/browser/rhymes/1.5

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=rhymes/1.4&new_path=rhymes/1.5

History:

Ticket Summary Status Resolution Owner
#46726 THEME: Rhymes – 1.5 closed live kafleg

(this ticket)

#48870 THEME: Rhymes – 1.6 closed live themetracbot


https://themes.svn.wordpress.org/rhymes/1.5/screenshot.png
Theme Check Results:

  • RECOMMENDED: 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.
  • RECOMMENDED: No reference to add_theme_support( "custom-background", $args ) was found in the theme. If the theme uses background images or solid colors for the background, then it is recommended that the theme implement this functionality.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • RECOMMENDED: Theme URI: is missing from your style.css header.

#18 in reply to: ↑ 16 @themescastle
4 weeks ago

Hello,

Sorry about that. Actually, the theme supports sticky header and while taking the screenshot it comes in between.

I have correct it and upload a new version.

Thanks.

#19 @kafleg
4 weeks ago

  • Resolution set to live
  • Status changed from reviewing to closed

No more issues found, setting live. Thank you so much for your coordination while reviewing theme.

Regards

Note: See TracTickets for help on using tickets.