search

lucaong / jQCloud

jQuery plugin for drawing neat word clouds that actually look like clouds
MIT License
646 stars 293 forks source link

More than one word cloud causes design to break #1

Closed seanahrens closed 13 years ago

seanahrens commented 13 years ago

Luca,

Thanks for the plugin. One issue I'm having is that the design of the clouds break if you try to do more than one word cloud on a page. Can you think of any quick workarounds or modifications to make to the .js file to fix this issue?

Thanks, Sean

amitklein commented 13 years ago

Same issue here

amitklein commented 13 years ago

hey i fixed this on my site... the problem is that it gets confused finding #word_index if there is more then one. i added an extra parameter #word1_index for the first time through and #word2_index for the next time. i can send you my fix if you'd like.

seanahrens commented 13 years ago

@amitklein, I just implemented a fix that allows for any arbitrary number of wordclouds per page. you can find it at https://github.com/seanahrens/jQCloud, and hopefully it'll be pulled into master shortly.

lucaong commented 13 years ago

pull request accepted. Issue should be fixed now. Thanks everyone :)

seanahrens commented 13 years ago

thanks luca, and thanks @amitklein (you pointed me to where the fix needed to be made!)