WordPress.org

Make WordPress Themes

#31909 closed theme (not-approved)

THEME: Community City – 1.0

Reported by: deblynprado Owned by: SheebaAbraham
Priority: new theme Keywords: theme-community-city
Cc: deblynprado@…

Description

Attachments (3)

Screen Shot 2016-09-18 at 8.45.26 AM.png (99.5 KB) - added by SheebaAbraham 13 months ago.
Search widget Sticking to the header
Screen Shot 2016-09-18 at 8.46.52 AM.png (38.9 KB) - added by SheebaAbraham 13 months ago.
Sticky template heading - sticking to the blue line of header
Screen Shot 2016-09-18 at 8.47.07 AM.png (150.5 KB) - added by SheebaAbraham 13 months ago.
Post text sticking to the borders of the post

Download all attachments as: .zip

Change History (16)

#1 @rinkuyadav999
14 months ago

Hi

Your theme will be start reviewing soon. so please make sure, you theme follow latest guideline:- https://make.wordpress.org/themes/handbook/review/required/

Note: Reviewers are allow to close ticket (theme), if more than 5 unique issues found. so please review your theme yourself and fix all "require" issues.

Thanks

#2 @leobaiano
14 months ago

Thank you, we have already seen the guideline and we believe that the subject agrees.

A doubt, if you can help. The theme was developed by 3 people, we include the authors name in style.css comments, thus:

Author: Deblyn Prado, Rafael Funchal, Leo Baiano

Thus even the statement?

#3 @themetracbot
14 months ago

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

#4 @SheebaAbraham
13 months ago

@deblynprado,

My review comments. Please do let me know if you need further clarifications. (Note this is still work in progress).

Look and feel
1) Tagline is not displayed.
2) Submenus not displayed as submenu.

No readme file.

Theme Check o/p
REQUIRED: This theme text domain does not match the theme's slug. The text domain used: commcitytheme. This theme's correct slug and text-domain is community-city.
WARNING: The tag responsive-layout has been deprecated, please remove it from your style.css header.
WARNING: The tag fixed-layout has been deprecated, please remove it from your style.css header.
WARNING: The tag dark has been deprecated, please remove it from your style.css header.
WARNING: The tag black has been deprecated, please remove it from your style.css header.

functions.php
L184 : You missed the stylesheet name in the script enqueue.
L204 : You can't override user's provided parameters. (Can't limit post to 3)

Review is in progress, I will add issues to the list.

Last edited 13 months ago by SheebaAbraham (previous) (diff)

#5 @leobaiano
13 months ago

We are already providing these early adjustments and wait for the next issues.

Thank @SheebaAbraham

#6 @SheebaAbraham
13 months ago

Look and feel
3) Theme is not responsive as mentioned in theme tags.
No readme file.

Recommended

  • Different function prefix used communitycity_ and city_
  • LICENSE should be with .txt


Review is in progress, I will add issues to the list.

@SheebaAbraham
13 months ago

Search widget Sticking to the header

@SheebaAbraham
13 months ago

Sticky template heading - sticking to the blue line of header

@SheebaAbraham
13 months ago

Post text sticking to the borders of the post

#7 @SheebaAbraham
13 months ago

@deblynprado,

My full review comments. Please do let me know if you need further clarifications.

Theme Check o/p

REQUIRED: This theme text domain does not match the theme's slug. The text domain used: commcitytheme. This theme's correct slug and text-domain is community-city.
WARNING: The tag responsive-layout has been deprecated, please remove it from your style.css header.
WARNING: The tag fixed-layout has been deprecated, please remove it from your style.css header.
WARNING: The tag dark has been deprecated, please remove it from your style.css header.
WARNING: The tag black has been deprecated, please remove it from your style.css header.

Look and feel

1) Tagline is not been displayed.
2) Submenus not displayed as submenu. 
3) Theme is not responsive, but Responsive is mentioned in theme tags. 
4) Sidebar sticks to the blue line, the few px of sidebar are not seen. (when the menu is two lines or more) - (See attached screenshot)
5) blog - the post text is sticking to the 3 boxes for post with image. (See attached screenshot) - Same with Sticky template heading.
 

functions.php

‘’’REQUIRED : ‘’’
L27: /language folder does not exist.
L178: functions.php - L163 Script handle bootstrap-js should be bootstrap.
L184 : You missed the stylesheet name in the script enqueue.
L204 : You can't override user's provided parameters. (Can't limit post to 3)

single.php

L10 : Do not override user’s preference of time format.

Other Recommended

  • Different function prefix used communitycity_ and city_
  • LICENSE should be with .txt
  • No readme file.
  • delete config.rb file

Outcome

I'm sorry, but the theme currently doesn't meet the requirement for approval. If this ticket is left as reviewing for 7 days, and it may be closed after that if there is no response. Note that if you need more time, please comment in the ticket - to avoid the ticket from getting closed.



Last edited 13 months ago by SheebaAbraham (previous) (diff)

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


13 months ago

#9 @deblynprado
13 months ago

Hi @SheebaAbraham !

Thanks for your review. We'll fix th theme and I'll let you know when we're ready.

:)

#10 @grapplerulrich
13 months ago

@deblynprado Please provide an update or the ticket will be closed.

#11 @leobaiano
13 months ago

@grapplerulrich we are working on the theme settings to update until the end of the week we should send a new file. All right?

#12 @grapplerulrich
13 months ago

@leobaiano A new week has started. How is the update coming on?

#13 @grapplerulrich
12 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.