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
Merged

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

rich-iannone added 30 commits Aug 14, 2019
* master:
  Fix for invisible top borders in rendered HTML tables (#359)
@rich-iannone rich-iannone requested review from schloerke and jcheng5 Sep 27, 2019
Copy link
Collaborator

@schloerke schloerke left a comment

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

Add an issue to fix rtf once merged.

@jcheng5
jcheng5 approved these changes Nov 8, 2019
Copy link
Member

@jcheng5 jcheng5 left a comment

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
3 of 5 checks passed
3 of 5 checks passed
continuous-integration/appveyor/branch AppVeyor build failed
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
license/cla Contributor License Agreement is signed.
Details
@rich-iannone rich-iannone deleted the dt_boxh branch Nov 8, 2019
@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
You can’t perform that action at this time.