WordPress.org

Make WordPress Themes

Opened 2 months ago

Last modified 3 days ago

#47041 reviewing theme

THEME: pagli – 1.0.5

Reported by: bnayawpguy Owned by: sakinshrestha
Priority: new theme Keywords: theme-pagli holiday
Cc: bnayawebguy@…

Description

pagli - 1.0.0

Description

Theme URL -
Author URL -

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.0

SVN - https://themes.svn.wordpress.org/pagli/1.0.0
ZIP - https://wordpress.org/themes/download/pagli.1.0.0.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.0/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

Attachments (2)

Theme Review – Just another WordPress site.png (33.6 KB) - added by sakinshrestha 10 days ago.
site title – taline.png (15.6 KB) - added by sakinshrestha 3 days ago.

Download all attachments as: .zip

Change History (13)

#1 @themetracbot
2 months ago

  • Summary changed from THEME: pagli – 1.0.0 to THEME: pagli – 1.0.1

pagli - 1.0.1

Description

Theme URL -
Author URL -

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.1

SVN - https://themes.svn.wordpress.org/pagli/1.0.1
ZIP - https://wordpress.org/themes/download/pagli.1.0.1.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=pagli/1.0.0&new_path=pagli/1.0.1

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.1/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

#2 @sakinshrestha
2 weeks ago

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

#3 @sakinshrestha
2 weeks ago

Hello @bnayawpguy,

Thanks for uploading theme, here are the list of issues you need to fix.

ISSUES:

  • Remove rtl.css if not used
  • Design issue: Logo overlaps header image
  • Quick question: How to hide default header image?
  • Got these error notification after activating Portfolio as Page template:
    NOTICE: ...\pagli\tpl-portfolio.php:49 - Trying to get property of non-object
       require('...\wp-blog-header.php'), require_once('...\wp-includes\template-loader.php'), include('...\wp-content\themes\pagli\tpl-portfolio.php')
     NOTICE: ...\wp-content\themes\pagli\tpl-portfolio.php:50 - Trying to get property of non-object
       require('...\review\wp-blog-header.php'), require_once('...\wp-includes\template-loader.php'), include('...\wp-content\themes\pagli\tpl-portfolio.php')
    
  • Remove option display social share button it falls under plugin terriotary
  • Footer credit link must be either Theme URL or Author URL so you need to add the link you have added in footer as Author Url

Note: Please communicate in ticket within 7days other wise theme will be resolved as not-approved.

#4 @themetracbot
11 days ago

  • Keywords holiday added
  • Summary changed from THEME: pagli – 1.0.1 to THEME: pagli – 1.0.2

pagli - 1.0.2

Description

Theme URL -
Author URL -

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.2

SVN - https://themes.svn.wordpress.org/pagli/1.0.2
ZIP - https://wordpress.org/themes/download/pagli.1.0.2.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=pagli/1.0.1&new_path=pagli/1.0.2

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.2/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

#5 @themetracbot
11 days ago

  • Summary changed from THEME: pagli – 1.0.2 to THEME: pagli – 1.0.3

pagli - 1.0.3

Description

Theme URL -
Author URL - http://paglithemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.3

SVN - https://themes.svn.wordpress.org/pagli/1.0.3
ZIP - https://wordpress.org/themes/download/pagli.1.0.3.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=pagli/1.0.2&new_path=pagli/1.0.3

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.3/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

#6 @bnayawpguy
11 days ago

Hi @sakinshrestha,
Thanks for taking your time and reviewing the theme. I have fixed the issues you have mentioned. Could you please check and confirm it.
Thanks,

#7 @sakinshrestha
10 days ago

Hi @bnayawpguy,

Thanks for the update, here are few more issues you need to fix:

ISSUES:

  • When header image is not added there is a gray space in place of header please check attached screenshot
  • Remove option Display social share if falls under plugin territory

#8 @themetracbot
5 days ago

  • Summary changed from THEME: pagli – 1.0.3 to THEME: pagli – 1.0.4

pagli - 1.0.4

Description

Theme URL -
Author URL - http://paglithemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.4

SVN - https://themes.svn.wordpress.org/pagli/1.0.4
ZIP - https://wordpress.org/themes/download/pagli.1.0.4.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=pagli/1.0.3&new_path=pagli/1.0.4

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.4/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

#9 @themetracbot
5 days ago

  • Summary changed from THEME: pagli – 1.0.4 to THEME: pagli – 1.0.5

pagli - 1.0.5

Description

Theme URL -
Author URL - http://paglithemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/pagli/1.0.5

SVN - https://themes.svn.wordpress.org/pagli/1.0.5
ZIP - https://wordpress.org/themes/download/pagli.1.0.5.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=pagli/1.0.4&new_path=pagli/1.0.5

History:

Ticket Summary Status Resolution Owner
#47041 THEME: pagli – 1.0.5 reviewing sakinshrestha

(this ticket)


https://themes.svn.wordpress.org/pagli/1.0.5/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

#10 @bnayawpguy
5 days ago

Hi @sakinshrestha,
I have fixed the issues you have mentioned.

About the Gray part being displayed, this part displays the title of the page, the title was initially in gray color which made it invisible. I have changed the title color for the pages now it seems to be displaying fine.
Thanks,

#11 @sakinshrestha
3 days ago

Hi @bnayawpguy,

Other issues fixed.

  • There is still design issue in sidebar while adding monster widget, I will attach the screenshot below
Note: See TracTickets for help on using tickets.