Carson Sievert

Results 294 comments of Carson Sievert

Thanks, if you could also update the `NEWS.md`, that'd be great!

Looking good to so far! I just changed file locations around a bit for better backwards-compatibility reasons. I also suggested a couple small tweaks, and left them unresolved in case...

Thanks for this impressive PR @earlygrey! Turns out, there's a few different scenarios in which `.recalculating` class doesn't get properly properly attached to recalculating output. #4039 aims to fix those...

Thanks @hq9000! It'd be helpful if you could run `devtools::document()` and push up the changes.

I think this was fixed in the latest release? https://github.com/rstudio/shiny/blame/main/NEWS.md#L44

> Mainly, I want the text labels to be readable, and not overlap each other. Sounds like your issues would be better handled at the ggplot2 level (not the graphics...

Thanks for reporting. Here's a more minimal reprex without `{flexdashboard}`. Also, since the error actually derives from Shiny's codebase, I'll be transferring the issue other there. ```` --- title: "thematic+renderPlot()...

Also, I'm sure we can avoid the error in this case, but I'm not sure whether we'll be able to get the "auto theming" behavior to work in the way...

> If the element does not exist, then it gets added to a javascript array of IDs idsToAttach that are pending (will likely appear later using insertUI/renderUI/etc) Seems like you...

> . But using setTimeout(..., 0) also does. It could be that the `setTimeout()` only happens to work for your test case. `taskQueue` should be more robust to network latency,...