WordPress.org

Make WordPress Themes

Opened 9 months ago

Closed 7 months ago

#40649 closed theme (not-approved)

THEME: Reduxbiz – 1.0.5

Reported by: pritamhans Owned by: benlumia007
Priority: previously reviewed Keywords: theme-reduxbiz
Cc: pritam.hans@…

Description

Reduxbiz - 1.0.3

Reduxbiz is an elegant and responsive theme perfect for your business’s online presence. Choose a custom color, add header logo & menu for your website easily. For business home page, activate the desired sections and customize each section with your own background image & text color scheme to create stunning website home page. Add your contact info, telephone & email links on home page easily. Add testimonials, team member & services details easily. With all the features for business websites, the theme also looks great as a personal blog.

Theme URL - http://reduxthemes.com/themes/reduxbiz/
Author URL - http://reduxthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/reduxbiz/1.0.3

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

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

History:

Ticket Summary Status Resolution Owner
#37987 THEME: Reduxbiz – 1.0.2 closed not-approved pratikshrestha
#40649 THEME: Reduxbiz – 1.0.5 closed not-approved benlumia007

(this ticket)


https://themes.svn.wordpress.org/reduxbiz/1.0.3/screenshot.jpg
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.

Change History (14)

#1 @themetracbot
9 months ago

  • Summary changed from THEME: Reduxbiz – 1.0.3 to THEME: Reduxbiz – 1.0.4

Reduxbiz - 1.0.4

Reduxbiz is an elegant and responsive theme perfect for your business’s online presence. Choose a custom color, add header logo & menu for your website easily. For business home page, activate the desired sections and customize each section with your own background image & text color scheme to create stunning website home page. Add your contact info, telephone & email links on home page easily. Add testimonials, team member & services details easily. With all the features for business websites, the theme also looks great as a personal blog.

Theme URL - http://reduxthemes.com/themes/reduxbiz/
Author URL - http://reduxthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/reduxbiz/1.0.4

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=reduxbiz/1.0.3&new_path=reduxbiz/1.0.4

History:

Ticket Summary Status Resolution Owner
#37987 THEME: Reduxbiz – 1.0.2 closed not-approved pratikshrestha
#40649 THEME: Reduxbiz – 1.0.5 closed not-approved benlumia007

(this ticket)


https://themes.svn.wordpress.org/reduxbiz/1.0.4/screenshot.jpg
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.

#2 @themetracbot
9 months ago

  • Summary changed from THEME: Reduxbiz – 1.0.4 to THEME: Reduxbiz – 1.0.5

Reduxbiz - 1.0.5

Reduxbiz is an elegant and responsive theme perfect for your business’s online presence. Choose a custom color, add header logo & menu for your website easily. For business home page, activate the desired sections and customize each section with your own background image & text color scheme to create stunning website home page. Add your contact info, telephone & email links on home page easily. Add testimonials, team member & services details easily. With all the features for business websites, the theme also looks great as a personal blog.

Theme URL - http://reduxthemes.com/themes/reduxbiz/
Author URL - http://reduxthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/reduxbiz/1.0.5

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=reduxbiz/1.0.4&new_path=reduxbiz/1.0.5

History:

Ticket Summary Status Resolution Owner
#37987 THEME: Reduxbiz – 1.0.2 closed not-approved pratikshrestha
#40649 THEME: Reduxbiz – 1.0.5 closed not-approved benlumia007

(this ticket)


https://themes.svn.wordpress.org/reduxbiz/1.0.5/screenshot.jpg
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.

#3 @pritamhans
9 months ago

Hi @djrmom @grapplerulrich ,

Request you to kindly move the existing theme & the following themes to @reduxthemes

https://wordpress.org/themes/prakashan/
https://wordpress.org/themes/undedicated/

Thanks for your support.

#4 @djrmom
9 months ago

@pritamhans, those 2 are set. This theme, I believe, has to be live before it can be moved. Thanks.

#5 @pritamhans
9 months ago

Thanks a tonne!

#6 @joyously
8 months ago

  • The menu needs to have a fallback for when no menu is selected. Remove the if statement around it.
  • Previously, regarding what @poena said about featured images just showing blue, you said

    Featured image: The featured image issue that you mention is most probably a local issue at your end

    • This is not correct. The styles are specifying a background image on the element (inline style) and a background color on a ::before psuedo-element of that same element, and since the psuedo-element is position:absolute it is out of the flow and is "over" the element. Just remove the blue background color, and it works fine, although using inline styles makes it much more difficult for a child theme to style it differently, and themes are supposed to be Child-Theme-Ready. There is also the Additional CSS option in Customizer to consider, and using inline styles are more difficult for the user to style differently.
  • The text that is over the featured images need just a little more text-shadow.
  • On a desktop screen, the submenu of the first menu item (left) is cut off. I know TwentySixteen had this issue and I fixed it for TwentySeventeen, but I don't remember right now how the fix works. You can look at the GitHub repository for it if you can't figure it out.
  • <q> tag is unstyled (default removed)
  • Page links (generated by <!--nextpage-->) are being output in the same div as the content instead of after it.
  • In Customizer, make it clear how to get the Home Page that is in the Home Page section. Perhaps rename it to "Home Page Template"?
  • Remove the Hero Banner section of Customizer, as that adds content that disappears on theme switch. Wouldn't you just use the featured image and page title here?
  • Remove the Call to Action and Contact Info section of Customizer, as that adds content that disappears on theme switch. These would be easier done with widget areas or in the actual page content. Maybe your Customizer option should be about where to place the actual page content relative to the other parts.
  • Remove the Contact Form section of Customizer, as that is plugin territory.
  • The only portions of the Services, Testimonials, and Team Members sections that are allowed is the selection of Pages. The rest is non-trivial content that disappears on theme switch. It would work a lot better for the user if the page template had several widget areas that are displayed with the actual page content, and the theme provides custom widgets that display pages and posts the way it is in the template. Then the user could choose the order to present the sections and which parts to include and how much in each part, even titles and images in the page itself. And the theme doesn't save content in its options!
  • Opinion: Really, you should consider what you are doing with this "everything on one page" type of template. It is bad marketing and bad for SEO. Good marketing and good SEO have a lot in common, one of which is "each page should have a focus". For marketing: a confused mind says No, so each page should be limited to "What is my most-wanted response?". For SEO: duplicate content issues (same content on multiple pages) and keyword dilution (focus keywords are diluted with too much other stuff on the page). So, your user might not know that, but you do, so it's better for them if you provide something good for their site instead of something that would hurt their marketing and SEO.
  • I tried a Print Preview of a category page and am confused by the results. My web page shows 5 posts, but the preview(4 print pages) shows the first three posts only. The preview has a mostly blank first paper page, 3 posts, and then footer widgets.
  • Pingback <link> should be conditional (singular and pings_open)

Please read the guidelines and use the theme test data as described here. Review tickets can be closed for having 5 or more required fix issues.

#7 @themetracbot
8 months ago

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

#8 @djrmom
7 months ago

  • Owner leroybuth 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.

#9 @benlumia007
7 months ago

@djrmom
If you have time, can you assign this ticket to me!

@pritamhans
I will be taking over this theme. Please give me 24 - 48 hours to complete a full review.

Thanks
@benlumia007

#10 @djrmom
7 months ago

  • Owner set to benlumia007

#11 @benlumia007
7 months ago

@pritamhans
I have completed your full review earlier, but I won't be able to post a comment till tomorrow. Thank for being patience.

Thanks
@benlumia007

#12 @benlumia007
7 months ago

  • testimonial needs to be remove as that is plugin territory.
  • index.php should be using the_posts_pagination(); instead of the_post_navigation();
  • the_posts_pagination(); is used for mulitple pages/posts and the_post_navigation(); is used for singe post.

#13 @pritamhans
7 months ago

Thank you all for your time to review this theme. Based on the suggestions, I will be updating the theme as there are some issues that got overlooked. Please close this ticket for now.

#14 @benlumia007
7 months ago

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

@pritamhans
Closing ticket as per author requests.

Note: See TracTickets for help on using tickets.