First option entry ignored for default search values per column

Open
  • Is something wrong?
#1,133 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
42/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
javascript, r
Domain
frontend

Research direction

Reproduce the issue with the provided DT::datatable example using filter = "top" and the searchCols values. Start by tracing how the R options are passed to the JavaScript DataTables widget, focusing on the first searchCols entry. Done means the first column displays the default value "col1" while subsequent columns retain their configured values.

Written by the indexing model from the issue text.

Description

This results in the default value of the first column Nr. 1 to be col2, instead of col1.

DT::datatable(
  cars,
  filter = "top",
  options = list(
    searchCols = list(
      list(
        search = "col1"
      ),
      list(
        search = "col2"
      ),
      list(
        search = "col3"
      ),
      list(
        search = "col4"
      )
    )
  )
)

image

R version 4.3.2 (2023-10-31 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 11 x64 (build 22631), RStudio 2023.12.1.402

Locale:
  LC_COLLATE=German_Switzerland.utf8  LC_CTYPE=German_Switzerland.utf8    LC_MONETARY=German_Switzerland.utf8
  LC_NUMERIC=C                        LC_TIME=German_Switzerland.utf8    

Package version:
  base64enc_0.1.3   bslib_0.6.1       cachem_1.0.8      cli_3.6.2         crosstalk_1.2.1   digest_0.6.35     DT_0.32          
  ellipsis_0.3.2    evaluate_0.23     fastmap_1.1.1     fontawesome_0.5.2 fs_1.6.3          glue_1.7.0        graphics_4.3.2   
  grDevices_4.3.2   highr_0.10        htmltools_0.5.7   htmlwidgets_1.6.4 httpuv_1.6.14     jquerylib_0.1.4   jsonlite_1.8.8   
  knitr_1.45        later_1.3.2       lazyeval_0.2.2    lifecycle_1.0.4   magrittr_2.0.3    memoise_2.0.1     methods_4.3.2    
  mime_0.12         promises_1.2.1    R6_2.5.1          rappdirs_0.3.3    Rcpp_1.0.12       rlang_1.1.3       rmarkdown_2.26   
  sass_0.4.9        stats_4.3.2       tinytex_0.50      tools_4.3.2       utils_4.3.2       xfun_0.42         yaml_2.3.8
Dominant language
JavaScript
Stars
621
Forks
177
No merged PRs in 30d

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from rstudio/DT

All issues in rstudio/DT

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.