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

Fix resolution of column labels when applying footnotes/styles #440

Merged
merged 4 commits into from
Dec 19, 2019

Conversation

rich-iannone
Copy link
Member

@rich-iannone rich-iannone commented Dec 13, 2019

This fixes the application of styles to column labels with the cells_column_labels() location helper in tab_style(). Internally, we now disregard hidden columns. Many testthat tests were added.

Fixes: #438
Fixes: #439

@rich-iannone rich-iannone marked this pull request as ready for review December 13, 2019 22:28
@rich-iannone rich-iannone requested a review from jcheng5 December 13, 2019 22:28
@rich-iannone rich-iannone merged commit ca4705f into master Dec 19, 2019
@rich-iannone rich-iannone deleted the cells_column_labels-fix branch December 19, 2019 17:56
rich-iannone added a commit that referenced this pull request Dec 19, 2019
* master:
  Fix resolution of column labels when applying footnotes/styles (#440)
rich-iannone added a commit that referenced this pull request Feb 5, 2020
* master:
  Remove use of emo package from example script (#469)
  Modify default CSS option for column `font-weight` (#468)
  Modify regex patterns to be less greedy
  Revise sentence in .Rmd file
  Revert changes to `as.tags` and add docs
  Add HTML dependency to the table
  Wrap HTML string in `HTML()`
  Place style block in <head>
  Fix for footnote glyphs not appearing with row labels (#447)
  Add test with `fmt_number()` and suffixing
  Fix #397: error with suffixing=TRUE with abs(x) < 1
  Perform input validation and check for types when formatting (#450)
  Expand documentation such that all functions get at least one self-contained example (#451)
  Fix resolution of column labels when applying footnotes/styles (#440)
  Renaming of API functions; documentation pass (#441)
  Implement fix for UTF-8 (#436)
  Modify implemention of fix and add tests
  Fix error with `fmt_number()` on an NA-only
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants