WordPress.org

Make WordPress Themes

Opened 3 years ago

Closed 3 years ago

#15833 closed theme (not-approved)

THEME: OpenStrapper - 1.5

Reported by: cooldude5757 Owned by: alex27
Priority: new theme Keywords: theme-openstrapper
Cc: cooldude5757@…

Description

OpenStrapper - 1.1

Open Source theme made using Twitter Bootstrap Framework. You are free to use it however you like.

Theme URL -
Author URL - http://www.facebook.com/jsksrinivas

SVN - http://themes.svn.wordpress.org/openstrapper/1.1
ZIP - http://wordpress.org/themes/download/openstrapper.1.1.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#15833 THEME: OpenStrapper - 1.5 closed not-approved alex27

(this ticket)


https://themes.svn.wordpress.org/openstrapper/1.1/screenshot.png

Attachments (1)

openstrapper.jpg (26.7 KB) - added by alex27 3 years ago.

Download all attachments as: .zip

Change History (20)

#1 @themetracbot
3 years ago

  • Summary changed from THEME: OpenStrapper - 1.1 to THEME: OpenStrapper - 1.2

OpenStrapper - 1.2

Open Source theme made using Twitter Bootstrap Framework. You are free to use it however you like.

Theme URL -
Author URL - http://www.facebook.com/jsksrinivas

SVN - http://themes.svn.wordpress.org/openstrapper/1.2
ZIP - http://wordpress.org/themes/download/openstrapper.1.2.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/openstrapper/1.1&new_path=/openstrapper/1.2

History:

Ticket Summary Status Resolution Owner
#15833 THEME: OpenStrapper - 1.5 closed not-approved alex27

(this ticket)


https://themes.svn.wordpress.org/openstrapper/1.2/screenshot.png

#2 @ZGani
3 years ago

The screenshot for the second upload is not acceptable - the first screenshot was fine.

Theme url is not acceptable as it is not accessible unless one has an account on and is logged in to Facebook.

The above need changing before the theme can be reviewed fully.

Please note that this is not a review but a pre-review advice note.

#3 @themetracbot
3 years ago

  • Summary changed from THEME: OpenStrapper - 1.2 to THEME: OpenStrapper - 1.3

OpenStrapper - 1.3

Open Source theme made using Twitter Bootstrap Framework. You are free to use it however you like.

Theme URL -
Author URL - http://www.convertgeek.com

SVN - http://themes.svn.wordpress.org/openstrapper/1.3
ZIP - http://wordpress.org/themes/download/openstrapper.1.3.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/openstrapper/1.2&new_path=/openstrapper/1.3

History:

Ticket Summary Status Resolution Owner
#15833 THEME: OpenStrapper - 1.5 closed not-approved alex27

(this ticket)


https://themes.svn.wordpress.org/openstrapper/1.3/screenshot.png

#4 follow-up: @cooldude5757
3 years ago

Can someone please tell me why my theme is not being listed in the wordpress community? I've worked hard on this one and I would be glad if this theme becomes useful for other wordpress users.

#5 in reply to: ↑ 4 ; follow-up: @ZGani
3 years ago

Replying to cooldude5757:

Can someone please tell me why my theme is not being listed in the wordpress community? I've worked hard on this one and I would be glad if this theme becomes useful for other wordpress users.

Your theme has not been reviewed yet!
Themes are required to pass the current theme requirements and regulation before they can be approved. Once approved one of the Admins will do a final check on the theme for completeness of the review before setting it to the live status.

Theme is in the que to be reviewed and one of us will review it soon - do bare in mind that it's the holiday season and it may take a little longer than normally would so be patient please.

Last edited 3 years ago by ZGani (previous) (diff)

#6 in reply to: ↑ 5 @cooldude5757
3 years ago

Replying to ZGani:

Replying to cooldude5757:

Can someone please tell me why my theme is not being listed in the wordpress community? I've worked hard on this one and I would be glad if this theme becomes useful for other wordpress users.

Your theme has not been reviewed yet!
Themes are required to pass the current theme requirements and regulation before they can be approved. Once approved one of the Admins will do a final check on the theme for completeness of the review before setting it to the live status.

Theme is in the que to be reviewed and one of us will review it soon - do bare in mind that it's the holiday season and it may take a little longer than normally would so be patient please.

Thank you for your quick reply, this is my first theme to be uploaded, I made changes as it has been requested, I am really interested in making more themes for the wordpress community, so I was a bit frustrated I didnt get proper attention. Merry christmas...

#7 @alex27
3 years ago

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

#8 follow-up: @alex27
3 years ago

Hello!

I've got two major issues before I can go through with full review:

  1. Look at the attached screenshot - this is how your theme looks after fresh install. You should provide some kind of default styling, now it looks like the website is broken.
  2. Authot URL - this is what the guidelines say:
    Author URI, if used, is required to link to an author’s personal web site or project/development website.
    For AuthorURI, emphasis is on the personal nature of the site.
    

Your Author URI doesn't seem to be a personal website.

@alex27
3 years ago

#9 in reply to: ↑ 8 @cooldude5757
3 years ago

Replying to alex27:

Hello!

I've got two major issues before I can go through with full review:

  1. Look at the attached screenshot - this is how your theme looks after fresh install. You should provide some kind of default styling, now it looks like the website is broken.
  2. Authot URL - this is what the guidelines say:
    Author URI, if used, is required to link to an author’s personal web site or project/development website.
    For AuthorURI, emphasis is on the personal nature of the site.
    

Your Author URI doesn't seem to be a personal website.

well, thank you for reviewing the theme I will make the changes you've asked and reupload it.

#10 @themetracbot
3 years ago

  • Summary changed from THEME: OpenStrapper - 1.3 to THEME: OpenStrapper - 1.4

OpenStrapper - 1.4

Open Source theme made using Twitter Bootstrap Framework. You are free to use it however you like.

Theme URL -
Author URL - http://profiles.wordpress.org/cooldude5757

SVN - http://themes.svn.wordpress.org/openstrapper/1.4
ZIP - http://wordpress.org/themes/download/openstrapper.1.4.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/openstrapper/1.3&new_path=/openstrapper/1.4

History:

Ticket Summary Status Resolution Owner
#15833 THEME: OpenStrapper - 1.5 closed not-approved alex27

(this ticket)


https://themes.svn.wordpress.org/openstrapper/1.4/screenshot.png

#11 @alex27
3 years ago

Hello!

Here's the full review of your theme:

REQUIRED

CODE

  • All stylesheets (other than style.css) and scripts should be enqueued via function hooked into wp_enqueue_script from functions.php. Please remove all hardcoded scripts and stylesheets from header.php
  • Themes are not allowed to load jQuery from external CDN or bundle jQuery with the theme. You must use version of jQuery bundled with WordPress instead. In fact you don't need to enqueue jQuery in your theme at all. It will be loaded anyway, since WordPress core uses it. Please remove jQuery script from js folder.
  • Calls to wp_nav_menu() should NOT include the 'menu' parameter. Please remove it.
  • All functions, class names, public and global variables are properly prefixed with your theme slug.
  • Theme options must be stored as a single array, should be sanitized on input, and escaped on output.
  • register_nav_menus should be included in bootstraptheme_setup() function.
  • All functions, class names, public and global variables are properly prefixed with your theme slug.
  • what are files pax_global_header and 1386340014.layoutit.html for?
  • _notes folder is empty. If it's not being used, you should remove it.

LAYOUT

  • Images are not properly aligned. You must properly style classes .alignleft, .alignright etc.
  • Very large images are overflowing container.
  • I noticed that in single post/page view there's no website header with website name. It makes it harder to navigate the website - just my thought.

OTHER

  • Credit link in footer must be either Author or Theme URI.
  • Recommended screenshot size is 880x660px (in general images with 4:3 ratio are acceptable).
  • You need to provide credit and license info for all resources bundled with the theme (fonts, scripts, libraries etc.), like so:

[Theme name] Uses [Resource Name and URL], licensed under [Name of license].
You can include this in readme.txt or style.css

  • Themes are required to declare copyright and license information as specified by the applicable license, e.g.:
    Themename WordPress theme, Copyright (C) 2013 YOUR NAME or YOUR COMPANY NAME
    Themename WordPress theme is licensed under the GPL.
    

This note should be included in style.css or readme.txt

  • If your theme is based on another theme (and yours seems to be based on _s starter theme), you should add a note indicating this in your readme.txt. Here's a good example that you can modify for your theme:

THEME NAME is based on the _s (Underscores) starter theme by Automattic Inc., licensed under LICENSE NAME.

CONCLUSION

Please fix above issues and upload new version of your theme, I'll leave this ticket open for another 2-3 days. Feel free to ask if you have any questions.

#12 @themetracbot
3 years ago

  • Summary changed from THEME: OpenStrapper - 1.4 to THEME: OpenStrapper - 1.5

OpenStrapper - 1.5

Open Source theme made using Twitter Bootstrap Framework. You are free to use it however you like.

Theme URL -
Author URL - http://profiles.wordpress.org/cooldude5757

SVN - http://themes.svn.wordpress.org/openstrapper/1.5
ZIP - http://wordpress.org/themes/download/openstrapper.1.5.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/openstrapper/1.4&new_path=/openstrapper/1.5

History:

Ticket Summary Status Resolution Owner
#15833 THEME: OpenStrapper - 1.5 closed not-approved alex27

(this ticket)


https://themes.svn.wordpress.org/openstrapper/1.5/screenshot.png

#13 follow-up: @tskk
3 years ago

Hello Alex,

Was reviewing http://themes.trac.wordpress.org/ticket/15645 and this ticket showed up, there is already a theme called "openstrap"

#14 in reply to: ↑ 13 @cooldude5757
3 years ago

Replying to tskk:

Hello Alex,

Was reviewing http://themes.trac.wordpress.org/ticket/15645 and this ticket showed up, there is already a theme called "openstrap"

Are you proposing me to change the name because there is another theme with ALMOST same name?
Clearly Openstrapper is different from openstrap. Just the way JQuery is different from JQueryUI.

#15 follow-up: @tskk
3 years ago

Yes, you will have to change the name.

#16 in reply to: ↑ 15 @cooldude5757
3 years ago

Replying to tskk:

Yes, you will have to change the name.

Hmm ok, Can I use OpenSwatch or TwitterSwatch?

#17 follow-up: @tskk
3 years ago

OpenSwatch would be better as twitter is trademarked.

#18 in reply to: ↑ 17 @cooldude5757
3 years ago

Replying to tskk:

OpenSwatch would be better as twitter is trademarked.

Done, uploaded it...
http://themes.trac.wordpress.org/ticket/15906

#19 @alex27
3 years ago

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

Ok, I'm closing this ticket.

Note: See TracTickets for help on using tickets.