WordPress.org

Make WordPress Themes

#14655 closed theme (live)

THEME: Adelle - 13.10

Reported by: bluchic Owned by: ZGani
Priority: previously reviewed Keywords: theme-adelle
Cc: hello@…

Description

Adelle - 13.09

WordPress theme by BluChic

Theme URL - http://www.bluchic.com/shop/adelle-theme
Author URL - http://www.bluchic.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/adelle/13.05&new_path=/adelle/13.09

History:

Ticket Summary Status Resolution Owner
#12210 THEME: Adelle - 13.03 closed closed-newer-version-uploaded
#12211 THEME: Adelle - 13.04 closed not-approved hchouhan
#12887 THEME: Adelle - 13.05 closed not-approved ZGani
#14655 THEME: Adelle - 13.10 closed live ZGani

(this ticket)

#15223 THEME: Adelle - 13.11 closed live emiluzelac
#18519 THEME: Adelle - 14.04 closed live jcastaneda
#21873 THEME: Adelle - 14.10 closed live karmatosed


https://themes.svn.wordpress.org/adelle/13.09/screenshot.png

Attachments (1)

ThickboxLoadsEntirePage.png (208.3 KB) - added by ZGani 18 months ago.
Thick should load only the image

Download all attachments as: .zip

Change History (11)

comment:1 @themetracbot18 months ago

  • Summary changed from THEME: Adelle - 13.09 to THEME: Adelle - 13.09.03

Adelle - 13.09.03

WordPress theme by BluChic

Theme URL - http://www.bluchic.com/shop/adelle-theme
Author URL - http://www.bluchic.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/adelle/13.09&new_path=/adelle/13.09.03

History:

Ticket Summary Status Resolution Owner
#12210 THEME: Adelle - 13.03 closed closed-newer-version-uploaded
#12211 THEME: Adelle - 13.04 closed not-approved hchouhan
#12887 THEME: Adelle - 13.05 closed not-approved ZGani
#14655 THEME: Adelle - 13.10 closed live ZGani

(this ticket)

#15223 THEME: Adelle - 13.11 closed live emiluzelac
#18519 THEME: Adelle - 14.04 closed live jcastaneda
#21873 THEME: Adelle - 14.10 closed live karmatosed


https://themes.svn.wordpress.org/adelle/13.09.03/screenshot.png

comment:2 @themetracbot18 months ago

  • Summary changed from THEME: Adelle - 13.09.03 to THEME: Adelle - 13.09.5

Adelle - 13.09.5

WordPress theme by BluChic

Theme URL - http://www.bluchic.com/shop/adelle-theme
Author URL - http://www.bluchic.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/adelle/13.09.03&new_path=/adelle/13.09.5

History:

Ticket Summary Status Resolution Owner
#12210 THEME: Adelle - 13.03 closed closed-newer-version-uploaded
#12211 THEME: Adelle - 13.04 closed not-approved hchouhan
#12887 THEME: Adelle - 13.05 closed not-approved ZGani
#14655 THEME: Adelle - 13.10 closed live ZGani

(this ticket)

#15223 THEME: Adelle - 13.11 closed live emiluzelac
#18519 THEME: Adelle - 14.04 closed live jcastaneda
#21873 THEME: Adelle - 14.10 closed live karmatosed


https://themes.svn.wordpress.org/adelle/13.09.5/screenshot.png

comment:3 @ZGani18 months ago

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

@ZGani18 months ago

Thick should load only the image

comment:4 follow-up: @ZGani18 months ago

  • Themes must not rely on external source (SVN) for required resources - All of the following items must be included and enqueued from with in the theme. These should be enqueued correctly and not hard coded in the head document.
    <!--[if lt IE 7]><script src="http://ie7-js.googlecode.com/svn/version/2.1(beta4)/IE7.js"></script><![endif]-->
    <!--[if lt IE 8]><script src="http://ie7-js.googlecode.com/svn/version/2.1(beta4)/IE8.js"></script><![endif]-->
    <!--[if lt IE 9]><script src="http://html5shim.googlecode.com/svn/trunk/html5.js"></script><![endif]-->
    <!--[if lt IE 9]><script src="http://css3-mediaqueries-js.googlecode.com/svn/trunk/css3-mediaqueries.js"></script><![endif]-->
    
  • Google fonts should be registered first before enqueueing.
  • adelle_theme_scripts() should be hooked to wp_enqueue_scripts() instead of wp_footer - this is the proper way of enqueueing scripts. Adding true to the array will automatically enqueue it in the footer.
  • Please remove if function exists wrapper from the following...
    add_image_size( 'post_thumb', 300, 200, true );
    add_theme_support( 'custom-background', array('default-color' => 'ffffff',) );
    if ( !function_exists('dynamic_sidebar')
    
  • Thickbox still does not function the way it should - it loads the entire page instead of the image. See attachment.

The above issues need fixing before the theme can be approved for inclusion in the repository.

comment:5 @themetracbot18 months ago

  • Summary changed from THEME: Adelle - 13.09.5 to THEME: Adelle - 13.09.8

Adelle - 13.09.8

WordPress theme by BluChic

Theme URL - http://www.bluchic.com/shop/adelle-theme
Author URL - http://www.bluchic.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/adelle/13.09.5&new_path=/adelle/13.09.8

History:

Ticket Summary Status Resolution Owner
#12210 THEME: Adelle - 13.03 closed closed-newer-version-uploaded
#12211 THEME: Adelle - 13.04 closed not-approved hchouhan
#12887 THEME: Adelle - 13.05 closed not-approved ZGani
#14655 THEME: Adelle - 13.10 closed live ZGani

(this ticket)

#15223 THEME: Adelle - 13.11 closed live emiluzelac
#18519 THEME: Adelle - 14.04 closed live jcastaneda
#21873 THEME: Adelle - 14.10 closed live karmatosed


https://themes.svn.wordpress.org/adelle/13.09.8/screenshot.png

comment:6 in reply to: ↑ 4 @bluchic18 months ago

Thank you very much for the guide! Update has been uploaded.

comment:7 @ZGani18 months ago

Just a couple of issues remain from above feedback.

You still have if function exist wrapper on the following...
functions.php

add_theme_support( 'custom-background', array('default-color' => 'ffffff',) );

sidebar.php - use if is_active ('dynamic_sidebar') instead of

if ( !function_exists('dynamic_sidebar')

In addition, favicons if implemented must off by default and be user configurable - please amend this or remove the implementation altogether.

comment:8 @themetracbot18 months ago

  • Summary changed from THEME: Adelle - 13.09.8 to THEME: Adelle - 13.10

Adelle - 13.10

WordPress theme by BluChic

Theme URL - http://www.bluchic.com/shop/adelle-theme
Author URL - http://www.bluchic.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/adelle/13.09.8&new_path=/adelle/13.10

History:

Ticket Summary Status Resolution Owner
#12210 THEME: Adelle - 13.03 closed closed-newer-version-uploaded
#12211 THEME: Adelle - 13.04 closed not-approved hchouhan
#12887 THEME: Adelle - 13.05 closed not-approved ZGani
#14655 THEME: Adelle - 13.10 closed live ZGani

(this ticket)

#15223 THEME: Adelle - 13.11 closed live emiluzelac
#18519 THEME: Adelle - 14.04 closed live jcastaneda
#21873 THEME: Adelle - 14.10 closed live karmatosed


https://themes.svn.wordpress.org/adelle/13.10/screenshot.png

comment:9 @ZGani18 months ago

  • Status changed from reviewing to approved

All required issues have been resolved.

comment:10 @chipbennett18 months ago

  • Resolution set to live
  • Status changed from approved to closed
Note: See TracTickets for help on using tickets.