Opened 7 months ago

Closed 5 months ago

Last modified 5 months ago

#9952 closed theme (not-approved)

THEME: Photologger - 1.0

Reported by: emmakitchen Owned by: Sanjiv Saha
Priority: major Keywords: theme-photologger
Cc: emma@…

Description

Photologger - 1.0

Photologger is a minimalist theme that is ideal for photo blogs, magazines & portfolios. Custom Background, Custom Header, Twitter, Facebook and Youtube links, Spacious, customiseable, clean, and very fast to load.

Theme URL - http://www.ekitchen.org.uk/photologger
Author URL - http://www.ekitchen.org.uk

SVN - http://themes.svn.wordpress.org/photologger/1.0
ZIP - http://wordpress.org/extend/themes/download/photologger.1.0.zip?nostats=1

All previous tickets for this theme: http://themes.trac.wordpress.org/query?keywords=~theme-photologger

http://themes.svn.wordpress.org/photologger/1.0/screenshot.png

Change History (6)

comment:1 Changed 6 months ago by Sanjiv Saha

  • Owner set to Sanjiv Saha
  • Status changed from new to accepted

comment:2 Changed 6 months ago by emmakitchen

Hi Sanjiv

Thanks for looking at my theme - if you need anything changed then let me know and I'll try my best to fix it!

Thanks again

Emma :)

comment:3 Changed 6 months ago by Sanjiv Saha

Theme Check Plugin: Pass
Deprecated Call Plugin: Pass
Theme Unit Test Data: Pass

Required

  • There is some error of the left footer sidebar coding. It is always showing Categories and Archives only although if other widgets are dragged into it. There is mismatch in the left footer sidebar id name. Fix it.
  • In style.css file there is line
    background:url(img/menu_divider.png) top right no-repeat;
    
    but there is no such image menu_divider.png in img folder.
  • In the functions.php file, there is
    load_theme_textdomain('photologger',get_template_directory() . '/languages');
    
    but I don't find languages folder inside your theme directory. Create it.
  • In the sinlge view of the post there is extra scroller inside the page. I don't see any use of it.

Suggestions

  • Comment every file before starting the coding part. Refer to twentytwelve theme.
  • In the home page view, if the post don't have featured image, there is no title of the post. There is only date and read more link. I find it a little ackward and the users will too. So, I suggest to show the the title as well.
  • I find the font used too small difficult for me to read as well.
  • The theme supports only one menu so rather use register_nav_menu than register_nav_menu.

@emmakitchen

I am keeping this ticket open. Please fix and upload new version and comment the ticket below. I will see it. :)

Sanjiv

comment:4 Changed 5 months ago by emmakitchen

Hi Sanjiv

Thanks for your comments - will try my best to fix these today, and will then upload the latest version of the theme for further feedback.

Thanks again

Emma :)

comment:5 Changed 5 months ago by Sanjiv Saha

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

comment:6 Changed 5 months ago by emmakitchen

Hi Sanjiv

Thanks for your help on this - and sorry for the delay, unfortunately I've have been in bed with flu.

I've just uploaded the amended version to http://themes.trac.wordpress.org/ticket/10340 - let me know if I've missed anything and I'll alter it as required.

Thanks again for your work on this, it is much appreciated.

Emma :)

Note: See TracTickets for help on using tickets.