WordPress.org

Make WordPress Themes

Opened 10 months ago

Closed 7 months ago

#38868 closed theme (closed-newer-version-uploaded)

THEME: The Mill – 1.0.0

Reported by: themeport Owned by: marvinkome
Priority: new theme Keywords: theme-the-mill
Cc: info@…

Description

The Mill - 1.0.0

The Mill is a classic WordPress theme that is perfect for your personal blog, business site, or online shop.

Theme URL - https://themeport.net/wordpress-themes/the-mill
Author URL - https://themeport.net/

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

History:

Ticket Summary Status Resolution Owner
#38868 THEME: The Mill – 1.0.0 closed closed-newer-version-uploaded marvinkome

(this ticket)


https://themes.svn.wordpress.org/the-mill/1.0.0/screenshot.jpg
Theme Check Results:

  • 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.
  • Warning: More than one text-domain is being used in this theme. This means the theme will not be compatible with WordPress.org language packs. The domains found are mill, woocommerce

Change History (9)

#1 @joyously
8 months ago

  • The Customizer option for Header Text Color could be more clearly stated as site tagline color. It does not show the correct default value.
  • Fatal error: Call to undefined function mill_post_title() in wp-content/themes/the-mill/index.php on line 27
  • Each item on the menu for smaller screens is centered. This disregards the nesting of the menu items, losing valuable organization of the site, and making it more difficult to find things.
  • The search results page really should use excerpts, so that the page is not unpredictably unwieldy and the user can quickly see the overview of where the keyword is used.
  • Let me just say, as a user, I really do not like sites with the same huge header on every page, forcing the reader to scroll. With my normal window size, I can only see the page title at the bottom of the window.

#2 @themeport
8 months ago

Hello,

Thank you for the review. I have made the necessary adjustments. The theme has been uploaded to a different account here: https://themes.trac.wordpress.org/ticket/40938

This version can be removed. Thank you.

#3 @themetracbot
7 months ago

  • Owner set to baikare.sandeep007
  • Status changed from new to reviewing

#4 @djrmom
7 months ago

  • Owner baikare.sandeep007 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.

New reviwer, you will need to continue the review in the new ticket https://themes.trac.wordpress.org/ticket/40938

#5 @themetracbot
7 months ago

  • Owner set to marvinkome

#6 @marvinkome
7 months ago

The new ticke, I'm not assigned to review it

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


7 months ago

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


7 months ago

#9 @greenshady
7 months ago

  • Resolution set to closed-newer-version-uploaded
  • Status changed from reviewing to closed

@marvinkome - I've assigned you the new ticket: https://themes.trac.wordpress.org/ticket/40938

Note: See TracTickets for help on using tickets.