Responsive Personal Blog is Drupal 7 theme. It is for the people who looking for their personal blog very neat and clean new standards in web 2.0.

The Responsive Personal Blog theme completely developed using HTML5 and CSS3 and jQuey. Theme is 100% compatible for all devises. Tested in FireFox, Google Chrome, Safari and Opera browsers and also tested in IE7+, Android, iPhone and iPad devises.

Download the theme using below git command:
git clone http://git.drupal.org/sandbox/parameshrag/1927798.git responsive_personal_blog

Features:

  1. Responsive, Mobile-Friendly Theme
  2. 2- column layout
  3. A total of 12 block regions
  4. 100% Mobile support (Smartphone, Tablet, iPhone, iPad and Android, etc)
  5. Banner content and Image Slideshow built with jQuery
  6. Original and unique theme design and nice typography
  7. HTML5 & CSS3 support clean mark-up
  8. CSS3 rounded corners supported in IE6, 7, 8 and 9. Tested in FireFox, Google Chrome, Safari, IE6, 7, 8 and 9, also Android, iPad, iPhone
  9. Facebook, Twitter, Delicious, Pinterest and other Social Icon.

Demo Site:

Sandbox url: http://drupal.org/sandbox/parameshrag/1927798

Comments

Ujval Shah’s picture

Hello Paramesh,

General Comments :

  1. You need to grab Review bonus
    Refer : http://drupal.org/node/1410826
  2. Drupal Coding Standards are missing.
    Please see : http://ventral.org/pareview/httpgitdrupalorgsandboxparameshrag1927798git
    Refer Manual : http://drupal.org/coding-standards
  3. It seems you have used bartik theme files. Please have a correct comments and information.
    Example : comment.tpl.php ; Line No : 5
    Bartik's theme implementation for comments.
    

Manual Review:

  1. DOCTYPE is not standard
    Example : html.tpl.php : line number - 1
           <!DOCTYPE html>
         

    better to use

           <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML+RDFa 1.0//EN" "http://www.w3.org/MarkUp/DTD/xhtml-rdfa-1.dtd">
         
  2. Better to use l() instead of direct href.
  3. Remove Inline CSS
    Example : page.tpl.php : line number - 125
           <section style="position: absolute; top: 0px; left: 0px; display: none; z-index: 4; opacity: 0; " id="slide-1">
         
  4. Use theme_image to populate the images
    Example : page.tpl.php : line number - 125
        <img src="<?php print base_path() . drupal_get_path('theme', 'responsive_personal_blog') . '/images/slide-01.png'; ?>" class="pngfix"/>
      
  5. In page.tpl.php, you have:
    if (module_exists('i18n')) with curly braces, better to use the alternative php syntax but even better is to take the logic out and put it into template.php somehow as a preprocess function. Refer "Alternate control statement syntax for templates" on http://drupal.org/coding-standards
  6. Remove un-necessary commented code:
    Example : page.tpl.php : line number - 66
         //print_r($node);
         //print $node->uid;
        
  7. In theme you have provided support for multilingual Menu, so better to provide consistency of same feature support to complete theme :
    Example : use t() for all text information.

UI Issues :

  1. Navigation Menu does not display correctly when Sub-items are defined and enabled as expanded. Find Attached Screenshot
  2. Cosmetic CSS changes for Cross Browser support needs some work.

Thanks,
Ujval Shah

mayank-kamothi’s picture

Hi,

Manul Review
:

You have quite a bit of logic in your template files (page.tpl.php for example). I would suggest anything that has theme_get_setting should go into template.php as a preprocess function perhaps and then create a variable that can be dropped into page.tpl.php.

  • In your region.tpl.php file comment is missing.
  • In featured.js file add Drupal.behaviors
  • In jquery_002.js all code indentation by tabs but as per drupal standard it should by space.
  • Once you enable and set this theme as default thema and login and your home page slider bottom navigation is display in overlay so you need to decrease your z-index.
  • In theme setting page Social Network Links setting section when i remove all social links ans save than "Be Social" block should be no loger to display but once i remove all this links still "Be Social block" is avaliable.

Mayank

mayank-kamothi’s picture

Status: Needs review » Needs work
StatusFileSize
new224.58 KB

Hi,
In node if comment is open and resize browser than there is no space between comment form and right sidebar block.

parameshrag’s picture

Status: Needs work » Needs review

Hi Ujval Shah and mayank,

Thank you very much for your review comments. I will work on the issues/standards and get back to you asap.

Regards,
Paramesh.

Homotechsual’s picture

For i18n menus when performing the logic please check for the presence of i18n_menu NOT i18n as the internationalization module can be installed without the menu component which will cause your theme to throw a fatal error.

danny englander’s picture

  1. The file jquery_002.js in your JS folder looks to be a version of JQuery (1.6.1?). If your theme is dependent on another version of JQuery, you should probably remove that file from the theme and suggest to your users to use the JQuery update module, otherwise including another version of JQuery in your theme could cause conflicts.
  2. There should also be a way to remove the footer copyright / author info / links in the theme settings, I am just not a big fan of having that in a theme that's contributed to drupal.org.
  3. You have CSS3 PIE included in your theme which is typically used for rounded corners in IE 8 and below but testing in IE8 yielded no rounded corners. It could be your path to PIE is wrong as I saw:
    sites/all/themes/param/css/PIE.htc
    ... which does not seem like a legit path and I have clean URLs enabled. I am also not sure if this will cause a conflict with users who are already using the Drupal CSS3 PIE Module
  4. You have a typo in your README -- "solial"

Other than that, it's a nice looking theme! -- Danny

danny englander’s picture

Status: Needs review » Needs work
manjit.singh’s picture

Hi,

I have review the theme in IE8, IE9, IE10, firefox, chrome, And found some UI bugs. Please see the following screenshots.

  1. I have comment on some random test page. After comment, layout for comment box disturbed. Refer screenshot(comments_UI_disturbed).
  2. Same thing is happening in preview comment. Refer screenshot(preview_comment).
  3. I am logged in with Admin, and open blocks and slideshow dots coming over the overlay. Issue is in IE,Chrome,Firefox. Refer screenshot(firefox,_chrome_UI_bug).
  4. I have create three blocks with some random text and show it in preface first, preface middle, preface last. In Firefox long text goes outside from the block. Issue is in Firefox only. Refer screenshot(firefox_block_text).
  5. Same blocks are disturbed in IE8. Only two blocks are shown, third block shift to lowerside. Refer screenshot(IE8_UI_disturbed).
  6. In this http://localhost/drupal7/?q=search/node/ link, search button is collapse with the field. Refer screenshot(search_button).
  7. The preface blocks are collapse with search block. Refer screenshot(search_collapse_with_blocks).
manjit.singh’s picture

StatusFileSize
new132.37 KB

Hi,

When i disable slideshow(at front page) from back-end, dots are coming. Refer screenshot.

PA robot’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

I'm a robot and this is an automated message from Project Applications Scraper.