CSS styles won't apply

Maxmyd - March 19, 2008 - 15:56
Project:Extra Voting Forms
Version:5.x-1.7
Component:User interface
Category:support request
Priority:normal
Assigned:Maxmyd
Status:closed
Description

Hello.

I use Up/Down voting form and everything I can see is a standard form with 2 checkboxes and the button. I tried to use default and fsdailys styles but no success. Can you please tell me what I'm doing wrong?
Thanks in advance.

AttachmentSize
vote.gif469 bytes

#1

mercmobily - March 20, 2008 - 01:39

Hi

Did you follow the installation instructions for extra_voting_forms _precisely_?
It looks like you don't have this in your page.tpl.php file:

<?php
if(module_exists('extra_voting_forms')){ print extra_voting_forms_insert_js_css(); }
?>

Please let us know,

Merc.

#2

Maxmyd - March 20, 2008 - 08:32

Yes I did that and I have those lines in the HTML source:

<style type="text/css" media="all">@import "/modules/extra_voting_forms/widgets/fsdailys/extra_voting_forms_all.css";</style>

<script type="text/javascript" src="/modules/extra_voting_forms/extra_voting_forms.js"></script>

Looks like I did everything what you have said in the install.txt. Maybe there is something else to do?

#3

Maxmyd - March 20, 2008 - 08:44

oops.... Sorry, my mistake. I didn't include following lines in the node.tpl.php:
<? if(module_exists('extra_voting_forms')){ print extra_voting_forms_show_form($comment, 'c', 1, 'big');} ?>
BTW IE7Pro plugin for Internet Explorer 7 won't show voting buttons images when its Ad filter is turned on. And this doesn't depend on widget you select (checked with default and fsdailys styles). Please consider this in next releases.

Thanks for your help

#4

mercmobily - March 20, 2008 - 09:07
Status:active» fixed

Hi,

OK closing this issue.
About this:

"BTW IE7Pro plugin for Internet Explorer 7 won't show voting buttons images when its Ad filter is turned on. And this doesn't depend on widget you select (checked with default and fsdailys styles). Please consider this in next releases."

This is a bug in IE7pro -- I suggest you submit it to the IE7Pro people.

Merc.

#5

Anonymous (not verified) - April 3, 2008 - 09:11
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

#6

Summit - May 27, 2008 - 13:08

Hi, For anybody who reads this. You need to add the:

<? if(module_exists('extra_voting_forms')){ print extra_voting_forms_show_form($comment, 'c', 1, 'big');} ?>

to every specific node-.tpl.php otherwise those nodetypes do not get the widget inserted. It looks abvious but it took me a couple of hours researching why the widget didn't come up on a certain nodetype.

greetings,
Martijn

#7

Prodigy - August 28, 2008 - 06:20
Status:closed» postponed (maintainer needs more info)

Latest Dev Version.

1.) Placed manually in node.tpl
2.) Did the jquery override (not big on this)
3.) 100% positive the content type was selected
4.) Ended up with the exact same as post 1 screenshot.

Where do you select the style you would like to use? e.g fsdaily, default, etc. Maybe that is what I'm missing?

#8

mercmobily - August 28, 2008 - 07:17

Hi,

Can you please give me your URL?
The style to use is set in Admin -> Settings -> Extra Voting Forms. We have recently changed that code -- I am wondering if there's a problem there. Please play with the optons and report back.

Merc.

#9

Prodigy - August 28, 2008 - 07:51

I'm using the dev version released 2 days ago. There is definitely no place in the admin to choose the style.

#10

mercmobily - August 28, 2008 - 08:07

Hi,

It's the very first option:

"Widget directory:"

The option has always been there. The checkbox below it is new.

Merc.

#11

Prodigy - August 28, 2008 - 08:15

I saw that, but I was under the impression that was if you wanted something other than the included 4 styles. "Default" is what I have in "Widget Directory" ... but the only thing being printed is the non javascript version (aka the 1st picture in this thread).

#12

mercmobily - August 28, 2008 - 08:46

Hi,

Please send me your site's URL. A temporary working login/pass pair would also be highly appreciated -- email it to merc followed by "at" and then mobily1.com.

Bye,

Merc.

#13

mercmobily - August 29, 2008 - 09:48

Hi,

Hello?

Merc.

#14

Prodigy - August 29, 2008 - 20:20

Sorry have been on a 2 day leave. It was on a test server, I just uninstalled the module due to frusturation, and having to replace my jquery file. I don't mind trying again though, I'm 99% sure I had done everything correct. I did have 5star displaying on the node as well however.

#15

mercmobily - August 29, 2008 - 21:17

Hi,

Please let me know if you are able to install the module on a public server with the problem showing, as this might help us find the bug.

If you're unable to do that, please close this bug... hopefully, if a bug is indeed there, somebody else will report it.

Bye,

Merc.

#16

Prodigy - August 30, 2008 - 08:23
Status:postponed (maintainer needs more info)» closed

#17

mercmobily - September 1, 2008 - 15:32
Status:closed» fixed

Hi,

Prodigy, you might want to know that this bug is now fixed.
To fix it, just change line 73 from:

< if ($widget_dir_themes===FALSE) {

To:

> if ($widget_dir_themes == FALSE) {

Bye,

Merc.

#18

Prodigy - September 2, 2008 - 20:22

Confirmed that this is fixed. : )

#19

mercmobily - September 2, 2008 - 21:19

Hi,

...really?

Merc.

#20

Prodigy - September 2, 2008 - 21:49

Hey Merc,

Yes , replacing the === w/ == seems to have fixed this. Good job Mate!

#21

mercmobily - September 2, 2008 - 21:57

Hi,

Ah... silly me, I am tired! :-D
I thought you were responding to a different issue...! :-D

Thanks for letting me know thought. And sorry for handling this so bloody porrly.

Merc.

#22

Anonymous (not verified) - September 16, 2008 - 22:02
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.