Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code not shown #52

Open
AnnekathrinSilvia opened this issue Apr 21, 2023 · 7 comments
Open

Code not shown #52

AnnekathrinSilvia opened this issue Apr 21, 2023 · 7 comments

Comments

@AnnekathrinSilvia
Copy link
Collaborator

The code modal is currently not show in the deployed server

@federicomarini
Copy link
Owner

I just checked and it also curiously doesn't work in the local versions either.
Maybe something changed in the way how a modalDialog is handled? OR did we change something and did not update that accordingly?

@federicomarini
Copy link
Owner

Seems to be a problem with the functionality of tippy.
If I remove the fact that the button has a tippy-like feature, this works as expected

I came to think it was that by removing all the "unnecessary things" to make a modal work.
And I "knew" it would work in the simple form as the "session info" modal was still working.

I liked the tippy hint, but we might consider dropping it if that makes the modal not appear as we want.

@federicomarini
Copy link
Owner

Ok, if we change e.g. this

fluidRow(
                  column(
                    width = 1,
                    tippy::tippy(
                      actionButton(inputId = "coder_emap_visnet",
                                   label = "",
                                   style = .helpbutton_biocstyle,
                                   icon = icon("user-edit")),
                      "Show the code for this plot",
                      placement = "right"
                    )
                  )
                )

to this

fluidRow(
                  column(
                    width = 1,
                    actionButton(inputId = "coder_emap_visnet",
                                   label = "",
                                   style = .helpbutton_biocstyle,
                                   icon = icon("user-edit")),
                    tippy::tippy_this(
                      elementId = "coder_emap_visnet",
                      tooltip = "Show the code for this plot",
                      placement = "right"
                    )
                  )
                )

it seems to work.
Maybe there is a way to change least code possible, I will look into it

@federicomarini
Copy link
Owner

Ok, seems with_tippy() instead of the simple tippy() calls we had can simply do it.
Checking on my end, will push.

Can you double check that too @AnnekathrinSilvia and @alpoplaw? You would need to install of course the github version for this (BiocManager::install("federicomarini/GeneTonic"))

@alpoplaw
Copy link

alpoplaw commented Apr 24, 2023 via email

@alpoplaw
Copy link

alpoplaw commented Apr 24, 2023 via email

@federicomarini
Copy link
Owner

After reinstalling - give a full restart of R at least?
There is that, and also the labels for the gene box

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants