jonocarroll/ggeasy

Grid lines

jonocarroll opened this issue ยท 8 comments

Could be something like easy_remove_gridlines(minor = TRUE, major = TRUE)

Which makes more sense to you?

easy_remove_x_gridlines(minor = TRUE, major = TRUE)

or

easy_remove_gridlines(minor_x = TRUE, major_x = TRUE)

or

easy_remove_x_minor_gridlines()

My suggestion would be to have these three functions:

  • easy_remove_x_gridlines(major = TRUE, minor = TRUE)
  • easy_remove_y_gridlines(major = TRUE, minor = TRUE)
  • easy_remove_xy_gridlines(major = TRUE, minor = TRUE) or alternatively easy_remove_all_gridlines(major = TRUE, minor = TRUE)

Sounds good - I'm fine with having lots of functions/aliases so both xy and all are fine by me.

Awesome! You'll get another pull request soon ๐Ÿ˜‰

Completed via #45

This is awesome :)