Bug fix: allow empty `Vlines` and `Hlines` as `ggplot2` does#2252

Merged

cpsievert merged 16 commits into

plotly:masterfrom

dereckmezquita:master

Jun 3, 2023

Commits on Apr 9, 2023

Vlines and Hlines might sometimes be inexistant; ggplot2 allows this but plotly fails on merging empty data.frame.
dereckdemezquita
committed
Unit test for allowing empty vlines.
dereckdemezquita
committed
Added horizontal line test.
dereckdemezquita
committed

Commits on Apr 11, 2023

If data is greater than nrow 0; better data check.
dereckdemezquita
committed
Simplified example data and corrected expected result in expect_error; will move code to test-ggplot-hline.R.
dereckdemezquita
committed
Tests with simplified data; did both cases succesful plot and what was previously failing.
dereckdemezquita
committed

Commits on Apr 15, 2023

Separated tests into respective files; horizontal/vertical.
dereckdemezquita
committed

Commits on Jun 3, 2023