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

[pull] master from talgalili:master #63

Open
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

pull[bot]
Copy link

@pull pull bot commented Aug 6, 2022

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label Aug 6, 2022
alanocallaghan and others added 21 commits March 22, 2023 14:41
This reverts commit 17dfbac.
To reproduce the issue, run:

```
heatmaply::heatmaply(mtcars, dendrogram = "row", plot_method = "plotly")
```

Notice that the row dendrogram has axis ticks at the bottom.
Hide row dendrogram axis ticks for plotly
* Add option suppress_default_hovertext

* Add suppress hovertext arg to ggplot_heatmap

* Style changes and invert if/else

* revert if/else switch

* Try fix for empty text slot with no custom and no default hovertext

* conform if style

* Add Matt Simenc to DESCRIPTION as contributor

* Fix DESCRIPTION, conform style, and improve consistency in logic

* Removed extra paren in DESCRIPTION file

* Added roxygen documentation for suppress_default_hovertext

* Add @param for suppress_default_hovertext to heatmapr

* Added item to roxygen Rd documentation for suppress_default_hovertext and modified param descriptions in R files to match.

* Remove @param for suppress_default_hovertext in heatmapr.R

* Ran roxygenise

---------

Co-authored-by: [email protected] <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants