Navigation bug

[sama55]

Thank you nice plugin. I found 2 issue.

1. qa-webmaster-layer.php (L50)

   $this->content['navigation']['main']['webmaster']['selected'] = true;
   // $this->content['navigation']['main']['selected'] = true; // << Unnecessary code

2. CSS of tab is not applied

[TohidN]

thanks for reporting these issues.

1. you are right, I removed it.
2. I just checked this plugin with 2 other themes, but it was styled correctly. can you please tell me where did you notice this issue?

[sama55]

I thank for your correspondence.
About issue 2.
There was not surely problem on the current theme. My theme is none standard HTML5. And, jquery script is linked in the last of body. Therefore, it was caused by the fact that your inline script became error. Q2A will shift to HTML5 in V1.7. In addition, script may be linked in the last of body at that time.

Example code which works on my HTML5 theme:

function head_script(){
    qa_html_theme_base::head_script();
    global $qa_request;
    if ( ($qa_request == 'webmaster') && (qa_get_logged_in_level()>=QA_USER_LEVEL_ADMIN) ) {
        $this->output('<script type="text/javascript" src="'. qa_opt('site_url') . $this->plugin_url .'include/easyResponsiveTabs.js"></script>');
        $this->output(
            '<script type="text/javascript">',
            '$("#verticalTab").easyResponsiveTabs({',
            '   type: "vertical",',      
            '   width: "auto",',
            '   fit: true',
            '});',
            '</script>'
        );
    }
}   
function body_footer(){
    global $qa_request;
    /*
    if ( ($qa_request == 'webmaster') && (qa_get_logged_in_level()>=QA_USER_LEVEL_ADMIN) )
        $this->output(
            '<script type="text/javascript">',
            '$("#verticalTab").easyResponsiveTabs({',
            '   type: "vertical",',      
            '   width: "auto",',
            '   fit: true',
            '});',
            '</script>'
        );
    */
    qa_html_theme_base::body_footer();
}

Thanks.

[TohidN]

ok, in next version I'll add a js file instead of inline code. I'm planning to add even more features to this plugin so I'll do them all at the same time.
thanks for contributing.