Leon Samson

Results 17 comments of Leon Samson

The idea is good. I would love to help with this if possible. Some thoughts below. One way to implement this change is: leave the original ggcorrplot function largely similar,...

See [here](https://github.com/LDSamson/ggcorrplot2/tree/development); this version has quite some changes since I split up the main functions in several smaller ones. Some of the changes (I can make a more detailed log...

Note that it is also possible to change the legend width and height within the theme() function: ``` theme(legend.position = ..., legend.key.height = .., legend.key.width =..) ``` @caijun would you...

I can understand that you want to keep the package as simple as possible, with not too many parameters in the function, I think that is a good design choice....

I tried to implement changes to address this issue [here](https://github.com/LDSamson/ggcorrplot2/tree/improve_labels). You can try this out locally by installing: ``` devtools::install_github("LDSamson/ggcorrplot2@improve_labels") ``` It is not perfect, but it has some changes...

See [this](https://github.com/plotly/plotly.R/issues/1114#issue-256325316) thread. I just posted a workaround function for this specific use case [here](https://github.com/plotly/plotly.R/issues/1114#issuecomment-1803541613). I think it is not yet solved in the current plotly version, but correct me...

> > > The problem is that when you also have `geom_jitter` in the plot (in addition to `geom_boxplot`), the `lapply` part will remove all the points. Is there a...

@cderv sorry, there was an error in the reprex. I fixed it in my initial post. Does it work for you now? The reprex should now hopefully demonstrate that it...

I don't know why the test-coverage GitHub Action it failing though..

A sidenote: if the card content is a bit more complex with nested divs using something like `bslib::layout_column_wrap()`, of which you need to only hide one element from, you will...