Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
theme
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
29 Nov 2012 at 17:31 UTC
Updated:
14 May 2013 at 10:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
monymirzaHi,
try to clear warnings and erros.
http://ventral.org/pareview/httpgitdrupalorgsandboxkaasi1817884git
You have also missed code standards. see here: http://drupal.org/coding-standards
Comment #2
isaakordonez commentedHi monymirza,
I fixed the errors,
Please Check: http://ventral.org/pareview/httpgitdrupalorgsandboxkaasi1817884git
Thank you very much.
Comment #3
monymirzatemplate.php
where it is
$form['actions']['submit'] = array('#type' => 'image_button', '#src' => base_path() . path_to_theme() . '/css/images/search-button.png');make proper like;
README.txt:
Add more description about your theme.
Comment #4
monymirzaComment #5
isaakordonez commentedI have only one error but i don't know how to solve it.
FILE: ...-7-pareview/sites/all/modules/pareview_temp/test_candidate/template.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
8 | ERROR | Missing function doc comment
--------------------------------------------------------------------------------
Can you help me please
Comment #6
monymirzaNot sure but try this,
Remove line #4 & #7.
Comment #7
isaakordonez commentedI fixed it
Please Check http://ventral.org/pareview/httpgitdrupalorgsandboxkaasi1817884git
Comment #8
monymirzaComment #9
cog-axis commentedHi Kassi,
Please review your theme from coder module it gives errors. See attachment. Remaining I have installed your theme will post you more on this.
Good luck :-)
Thanks!
Comment #10
cog-axis commentedHi Kassi,
Please review your theme from coder module it gives errors. See attachment. Remaining I have installed your theme will post you more on this.
Good luck :-)
Thanks!
Comment #11
cog-axis commentedSorry about duplicate message. See attached image.
Comment #12
monymirzaComment #13
mayank-kamothi commentedHi,
Manual Review:
I have checked your theme and resize browser and found some issue ,I have attached screen-shot.
issue-1.png: Search button issue when resize browser
issue-2.png: When resize browser slider navigation not align proper and between "About Inspira" and naviagtion control have no space.look like both are joins.
Mayank Kamothi
Comment #14
PA robot commentedClosing 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.