WordPress.org

Make WordPress Themes

Opened 3 years ago

Closed 3 years ago

#20699 closed theme (not-approved)

THEME: Biancaa - 1.0.0

Reported by: themejunkie Owned by: pratikshrestha
Priority: new theme Keywords: theme-biancaa
Cc: guanfulai@…

Description

Biancaa - 1.0.0

BiancaA is a premium like, mobile ready, beautiful and easy-to-use Photoblogging WordPress theme. Perfect for professional blogger and photoblogger.

Theme URL - http://www.theme-junkie.com/themes/biancaa/
Author URL - http://www.theme-junkie.com/

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

History:

Ticket Summary Status Resolution Owner
#20699 THEME: Biancaa - 1.0.0 closed not-approved pratikshrestha

(this ticket)

#22456 THEME: Biancaa - 1.0.4 closed live chemiker
#27071 THEME: Biancaa – 1.0.5 closed live karmatosed
#40680 THEME: Biancaa – 1.0.6 closed live themetracbot


https://themes.svn.wordpress.org/biancaa/1.0.0/screenshot.png


Change History (4)

#1 @emiluzelac
3 years ago

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

#2 @pratikshrestha
3 years ago

Theme Check: No issues
Including Stylesheets & Scripts: No issues
Plugin Territory: No issues


Code Quality:

  • $content_width definition needs to be inside a function scope and globalized. Preferably, move it inside theme setup function.

Theme Security and Privacy:

  • Use esc_url_raw() to sanitize input links instead of esc_url() and need to escape social links using esc_url() before output in widget-about.php.
  • Escape the_author_meta social urls in about.php.
  • Need to sanitize all add_setting() using a 'sanitize_callback' parameter in customizer. Missing in a few.

License, Theme Name, Credit Links, Up-Sell Themes

  • Themes are required to state the copyright and license information for any bundled resources (e.g. PHP, CSS, JS, fonts and images) not covered by the Theme’s license statement.
  • Themes are required to state the copyright and license information for any bundled resources not covered by the Theme’s license statement. Themes are recommended to state this information in the Theme’s README documentation.
    • Need correct attributions for bundled resources like hybrid framework, html5shiv, genericons, etc.
  • Need to mention license details of the resources (images) used in the screenshot as well.

Recommended:

  • Themes are recommended to include a readme.txt file, using Plugin readme.txt markdown.

Please let me know if there are any confusions or issues with the review. We can discuss it further.

Thanks,
Pratik

This ticket was mentioned in Slack in #themereview by pratikshrestha. View the logs.


3 years ago

#4 @pratikshrestha
3 years ago

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

Closing the ticket as there is no reply from author.

@yhemejunkie: If you update new version , you can give the ticket number here and i can resume the review.

Note: See TracTickets for help on using tickets.