WordPress.org

Make WordPress Themes

#29635 closed theme (not-approved)

THEME: Birva – 2.1

Reported by: ckthemes Owned by: kkoppenhaver
Priority: previously reviewed Keywords: theme-birva
Cc: weblogixindia@…

Description

Change History (16)

#1 @themetracbot
18 months ago

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

#2 @kkoppenhaver
18 months ago

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

Hello,

Did a pretty thorough front end review and found a few issues.

General:

  • Theme Screenshot should be 1200x900 so that it works well on larger displays.
  • Look into using https://codex.wordpress.org/Theme_Unit_Test to check your theme for many of the cases I'll cover below.
  • Documentation should be added as to what features are available through the customizer and how to get the homepage looking like the screenshot

Homepage:

  • Twitter section adds 'http://' to the twitter username, and results on a link like https://twitter.com/http://kkoppenhaver when you click on it. As a result, latest tweets are also not pulled in.
  • Added documentation on which features are located in the customizer would be helpful. I spent a reasonable amount of time messing with the homepage before finding many of the options that I was looking for were in the customizer.
  • After populating a nav menu, the items are hidden on the homepage. This looks to be an alignment issue. If I tweak the CSS, I can get the items to show, but items should show properly on theme activation. If there is a restriction on how many items can be added or if items have to be added a certain way, that needs to be documented.
  • Initial homepage design does not match the theme screenshot.

Other Pages:

  • In general, images stretch their containers. Vertically, the containers are stretched on the blog archive page and when images are too large horizontally, they simply overflow their containers.
  • On post pages, the sidebar items' containers are squished around them. .widget li a is currently floated left. If you remove this float, then they sit in their containers properly.

Takeaways:

  • Please update and add to the documentation so that we can more closely replicate what you have posted as your theme screenshot.
  • Please make sure a variety of different content types are being displayed correctly. https://codex.wordpress.org/Theme_Unit_Test. If your theme is not meant to handle all these different types, please document that clearly so we know for future testing.

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


18 months ago

#4 @joedolson
18 months ago

  • Resolution not-approved deleted
  • Status changed from closed to reopened

#5 @joedolson
18 months ago

Ticket was closed by accident - now re-opened!

#6 @kkoppenhaver
18 months ago

@ckthemes: I've completed an initial review. See my comments above.

#7 @grapplerulrich
18 months ago

  • Status changed from reopened to reviewing

#8 @ckthemes
18 months ago

Thanks We get review comment and make it solve and update theme asap

#9 @ckthemes
17 months ago

we are in the process of removing review error so please do not close our theme review ticket as we fixed all issues we will update theme.

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


17 months ago

#11 @kkoppenhaver
17 months ago

@ckthemes Any update on this?

#12 @ckthemes
17 months ago

Hi,

We are working on issues and will submit by 31st May.

I have one query .. we have on theme in wordpress.org name is "kanu"

url : https://wordpress.org/themes/kanu/

My question is that we don't find "Theme Homepage" url below preview download button. can you please let me know what is the issue why "Theme Homepage" link not coming in "kanu" theme

Thanks
CK Themes

#13 @kkoppenhaver
17 months ago

I'm assuming it's because you don't have a Theme URI defined in style.css

See Theme File Header Example here: https://codex.wordpress.org/File_Header

#14 @grapplerulrich
17 months ago

@ckthemes It is June 2nd now...

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


17 months ago

#16 @grapplerulrich
17 months ago

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

Closing due to no response.

Note: See TracTickets for help on using tickets.