ggplot VI cant handle varied coordinate systems

[1jamesthompson1]

Currently, the VI method will only work if the plots coordinate first calls is either "CoordCartesian" or "CoordFixed".

This means that a plot like:

ggplot(mtcars, aes(mpg,disp)) +
  geom_point() + coord_flip()

Will print out a message like this:

VI cannot process ggplot objects with flipped or non-Cartesian coordinates

I think that the VI method is actually capable of processing different types of Coord systems like flipped with ease.

It seems that

  if (!(.getGGCoord(x, xbuild) %in% c("CoordCartesian", "CoordFixed"))) {
    message("VI cannot process ggplot objects with flipped or non-Cartesian coordinates")
    return(NULL)
  }

At the start of .VIstruct.ggplot() and .getGGCoord() need to be updated.

[ajrgodfrey]

Sounds reasonable. You’ll see the use of message() in files (other than Messages.R); take them as an indication that it is something I’d like to see improved somehow.