I think we need:
in faq_access:
if ($op == 'view') {
return user_access('access content')
}
Line 53
$items[] = array('path' => 'admin/content/faq',
Line 68
'path' => 'admin/content/faq/configure',
Line 180
grammar change (not "brings")
$display_options['questions_top'] = t('Clicking on question takes user to answer further down the page');
Line 187
typo error
'#description' => t('This controls how the questions and answers are displayed on the page and what happens when someone clicks on the question.'),
Line 347, 390
would prefer to have css classes should be faq specific
$indent = '
- ';
Lines 395-405
would prefer to have consistency in - or _ in css names
$header = '
' . t($term->name) . '
';
Comments
Comment #1
nancydruThis messed up:
Lines 395-405
would prefer to have consistency in - or _ in css names
$header = '<h5 class="faq_qa_header">' . t($term->name) . '</h5>';Comment #2
stella commentedHi,
Thanks for the feedback! I've committed the changes to css, so if you take faq.css and faq.module from HEAD in CVS, you should have the latest version to play around with. Most of the changes were minor cosmetic changes so I won't be doing another release until more people have had a chance to test it.
Cheers,
Stella
Comment #3
stella commentedComment #4
stella commentedreleased in 5.x-1.1