Skip to content

[pull] master from talgalili:master#63

Open
pull[bot] wants to merge 59 commits intoHBPMedical:masterfrom
talgalili:master
Open

[pull] master from talgalili:master#63
pull[bot] wants to merge 59 commits intoHBPMedical:masterfrom
talgalili:master

Conversation

@pull
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 30 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: mcsimenc@gmail.com <mcsimenc@Mathews-MacBook-Air.local>
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