Lchiffon / wordcloud2

R interface to wordcloud for data visualization.
397 stars 113 forks source link

Hover does not work with mulitiple wordclouds. #18

Open jahagirdar opened 7 years ago

jahagirdar commented 7 years ago

Example http://data.dyumnin.com/analysis/tweeplytics/ashutosh83B.html This page has 4 wordclouds, If we mouse over the words on any of the wordclouds the rectangular frame and word count appears over the first cloud instead of the cloud over which the mouse is positioned.

AdeelK93 commented 7 years ago

Would you happen to have the code used to make this? I see what you're describing in the link, but I'm unable to reproduce it with multiple wordclouds on my own

jahagirdar commented 7 years ago

Can you try with this https://github.com/DataDrivenPolicy/twitteralytics/blob/master/src/twitter_profiles.Rmd

AdeelK93 commented 7 years ago

Do you have the other associated files with this Rmd? Some of the required csv's don't seem to be included. Or better yet, would you happen to have a more minimal example?

AdamSpannbauer commented 6 years ago

Below is a minimal example of the hover bug. The 2 wordclouds in the fluidRow exhibit the bug (ie the hover info works fine for "my_wc" but the hover info for "my_wc2" is displayed on top of "my_wc"). The 2 wordclouds placed directly into fluidPage have no issues with the hover information.

library(shiny)
library(wordcloud2)
shinyApp(
  ui=shinyUI(fluidPage(
    fluidRow(
      column(width=6,align="center", wordcloud2Output("my_wc")),
      column(width=6,align="center", wordcloud2Output("my_wc2"))
    ),
    wordcloud2Output("my_wc3"),
    wordcloud2Output("my_wc4")
  )),
  server=shinyServer(function(input,output,session){
    output$my_wc  = renderWordcloud2(wordcloud2(demoFreq))
    output$my_wc2 = renderWordcloud2(wordcloud2(demoFreq))
    output$my_wc3 = renderWordcloud2(wordcloud2(demoFreq))
    output$my_wc4 = renderWordcloud2(wordcloud2(demoFreq))
  })
)
AdeelK93 commented 6 years ago

ezpz #32

That look alright to you @AdamSpannbauer?

AdamSpannbauer commented 6 years ago

That seems to be fix the issue on my end. Thanks @AdeelK93

Tested on linux box w/R 3.4.1 & windows box w/R 3.4.2 using Chrome for browser.

duocang commented 6 years ago

I am still confused about this issue, especially in a shiny module. I create a shiny module to implement wordcloud2, and I call this module itself inside the module to create a modal(pop up window). The hover text does not work(just like http://data.dyumnin.com/analysis/tweeplytics/ashutosh83B.html)

I only give the abstract of my code, since it has 800 lines for one module. You donot want to read it. namesapce might be the problem for this issue.

I only can use CARN version. So, I do not if #32can solve this problem or not.

##################################################################
\# UI
##################################################################
keywordWordcloudInput <- function( id, title = "Keywords"){
   ......
   ......
}

########################################################################
\# Server
#######################################################################
keywordWordcloud <- function( input,  output, session,data.keywords){
    ....
    ....
    #
    # Full screen modal
    # 
    modalUI <- function(){
      # Use namespace provided by session
      ns <- session$ns

      # UI content
      modalDialog({ 
            .......
            # Body
            fluidRow(
                 keywordWordcloudInput(
                 id = ns("keywordWordclouf_fullscreen")
            )#fluidRow
       })#modalDialog
     }#modalUI

     callModule(
        keywordWordcloud,
        id = "keywordWordclouf_fullscreen",
        reactive(data.keywords()),
        fullscreen_link = FALSE
      )#callModule

}
AdamSpannbauer commented 6 years ago

Do you still have the issue if you install #32 ?

duocang commented 6 years ago

sorry to mention, i can not install developer version. i will change my question. I really like wordclod2, but if I can not slove the issue in CRAN version then i have to give it up.

Thank you.

On Tue, 9 Jan 2018 at 12:36, Adam Spannbauer notifications@github.com wrote:

Do you still have the issue if you install #32 https://github.com/Lchiffon/wordcloud2/pull/32 ?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Lchiffon/wordcloud2/issues/18#issuecomment-356259917, or mute the thread https://github.com/notifications/unsubscribe-auth/ANfdN5hIE3_ufkNneGA-BggQn2UtgYkOks5tI09TgaJpZM4Lmh_5 .

AdamSpannbauer commented 6 years ago

If it does fix your issue I hope that it can serve as more motivation for @Lchiffon to merge the PR. I'm also prevented from using wordcloud2 until a fix for the issue is on CRAN.

duocang commented 6 years ago

I just checked, wordcloud2 was updated on 3rd Jan 2018. So I assume it has been fixed. I will check it at work tomorrow and let you guys know.

On 9 January 2018 at 13:54, Adam Spannbauer notifications@github.com wrote:

If it does fix your issue I hope that it can serve as more motivation for @Lchiffon https://github.com/lchiffon to merge the PR. I'm also prevented from using wordcloud2 until a fix for the issue is on CRAN.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Lchiffon/wordcloud2/issues/18#issuecomment-356275930, or mute the thread https://github.com/notifications/unsubscribe-auth/ANfdN_Yk6jUf-lPBJlL4_5KRCp-INKl9ks5tI2FygaJpZM4Lmh_5 .

AdamSpannbauer commented 6 years ago

The updated version on CRAN might fix your issue.

For the issue of hover with multiple wordclouds it made the issue worse. The minimal example posted earlier in this thread only plots 1 of the 4 wordclouds with the version 0.2.1 on CRAN.

AdeelK93 commented 6 years ago

@wangxuesong29 Try using this version of wordcloud2: https://github.com/AdeelK93/wordcloud2

The CRAN version does not include #32