deblynprado / community-city Goto Github PK

Actually we're importing Bootstrap in theme but we just using the grid of Bootstrap.

We can make your own grid via SASS/Compass or customize the Bootstrap file to load only the grid and nothing more.
What's the best option in this case?

We already have support for it in functions.php but we need to insert it in header.php (only in home)

Codex: http://codex.wordpress.org/Custom_Headers

Refs: #5

In style.css define the theme tags, these tags are used to indicate to users the type of layout and the features it has. Tags below are depreciated then it is interesting to remove them.

responsive-layout
fixed-layout
dark
black

Add Grunt task to generate the final package.

Show comments in posts and pages

Need to create a language folder and create its respectives files

It must include the feedback form on the single page.

You must display the tagline somewhere theme, my suggestion is that it appears just below the title, in the site header, as is standard in most subjects.

We need to show the menu, second-level items. Currently the submenu is not displayed when you place the mouse over the parent item.

This theme doesn't seem to display tags. This is required and was appointed in theme checker.

Add wp_link_pages in single

Currently the site title is static, as you can see in this line:

https://github.com/deblynprado/community-city/blob/master/header.php#L36

We should make it dynamic to send it to the themes repo.

Found the URL of a CDN in the code: maxcdn.bootstrapcdn.com/bootstrap. You should not load CSS or Javascript resources from a CDN, please bundle them with the theme.

The text domain we are using the theme "commcitytheme" but the text domain should match the theme of the slug. The theme name is Community City, the slug is community-city, so the text domain should be community-city.

We must look at all the files, including style.css because the initial comments it has the text tag domain

In style.css define the theme tags, these tags are used to indicate to users the type of layout and the features it has. The fixed-layout tag has been deprecated so it's interesting now remove it.

The sticky classes, screen-reader-text and post author identified as necessary by the checker themme.

Look and feel, point 3 - Theme is not responsive, but Responsive is mentioned in theme tags.

Need to take a look and fix possible responsiveness issues.

In style.css define the theme tags, these tags are used to indicate to users the type of layout and the features it has. The responsive-layout tag has been deprecated so it's interesting now remove it.

Between the lines 201 and 209 of the functions.js there is thecity_limit_posts_per_page ()which is the callback action" pre_get_posts ". This feature sets it on the home page returns only 3 posts ignoring the sticky, according to the evaluation can not modify user settings, ie the amount of posts displayed should be a user setting via the control panel.

My suggestion is to remove the entire function and leave the pattern loop at home, this solves the problem, you only need to test to see if that will not break the layout when it has more than 3 posts, but if you need to adjust the CSS.

I've noticed that now the header image is displayed in all pages, not only in home.

As it's inserted in CSS, we should fix it.

The theme of the tags defined in style.css header should follow the pattern set in themme tags of Themme Review Team. It is necessary to reformulate the tags.

Add post_class the_ID for id in container post

Look and feel point 5 - blog - the post text is sticking to the 3 boxes for post with image. (See attached screenshot) - Same with Sticky template heading.

On line 184 of functions.js carry the main CSS file and use the name "odin-style",need to change the name to the theme of the slug, ie "community-city "

Non-printable characters were found in the category.php file. You may want to check this file for errors.

Look and feel point 4 - Sidebar sticks to the blue line, the few px of sidebar are not seen. (when the menu is two lines or more)

We need re-write SASS to use the default classes of WordPress.

Replace bloginfo function ('url') by esc_url (HOME_URL ()) in header.php line 36