Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dt_*() methods for all major, internal data structures #355

Merged
merged 103 commits into from
Nov 8, 2019

Conversation

rich-iannone
Copy link
Member

@rich-iannone rich-iannone commented Aug 15, 2019

This PR provides a suite of functions (e.g., dt_boxh_*(), dt_spanners_*(), etc.) for initializing, setting (as attrs), and modifying internal objects.

Fixes: #381
Fixes: #367
Fixes: #360
Fixes: #356
Fixes: #317
Fixes: #298
Fixes: #266
Fixes: #264
Fixes: #242
Fixes: #228
Fixes: #214
Fixes: #99
Fixes: #23

schloerke
schloerke previously approved these changes Sep 27, 2019
Copy link
Collaborator

@schloerke schloerke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is known to break rtf output, but given the major improvements, that is ok.

Add an issue to fix rtf once merged.

Copy link
Member

@jcheng5 jcheng5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't actually review this, consider this a resurrection of @schloerke's Approval.

@rich-iannone rich-iannone merged commit 48f2d73 into master Nov 8, 2019
@rich-iannone rich-iannone deleted the dt_boxh branch November 8, 2019 20:17
@rich-iannone rich-iannone mentioned this pull request Nov 8, 2019
rich-iannone added a commit to batpigandme/gt that referenced this pull request Nov 11, 2019
* master: (59 commits)
  add fmt_markdown to _pkgdown.yml (rstudio#371)
  Use `tidy_grepl()` instead of `expect_match()`
  Use `tidy_grepl()` & `expect_true()`
  Add `dt_*()` methods for all major, internal data structures (rstudio#355)
  Update CLA text (rstudio#373)
  remove reliance on Github version of sass, which is now updated on CRAN (rstudio#366)
  Fix for invisible top borders in rendered HTML tables (rstudio#359)
  CSS modifications (rstudio#346)
  Incorporate alpha option to `cell_fill()` (rstudio#340)
  Enable option to set row group padding in `tab_options()` (rstudio#342)
  Modify `filter()` statement (rstudio#341)
  Allow use of integers in `pct()`/`px()` (rstudio#336)
  Fix for resolving row group cells (rstudio#335)
  Easier access to footnote mark options (rstudio#333)
  Export `as.tags.gt_tbl()` (rstudio#331)
  Several `rlang`-based fixes (rstudio#334)
  Add function to specify column widths: `cols_widths()` (rstudio#293)
  Add `rstudio/sass` to Remotes list (rstudio#326)
  Fix for `cell_borders()` (rstudio#323)
  Allow an empty table with zero rows to create an empty gt table (rstudio#283)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment