Skip to content

Conversation

gadenbuie
Copy link
Collaborator

@gadenbuie gadenbuie commented Mar 8, 2024

Previously, we were setting web_component(style=) directly, preventing users from providing their own style argument. This PR adjusts how we set our own inline styles so that it won't conflict with user styles.

@gadenbuie gadenbuie requested a review from schloerke March 8, 2024 17:53
@gadenbuie gadenbuie marked this pull request as ready for review March 8, 2024 17:53
@gadenbuie gadenbuie requested a review from cpsievert March 19, 2024 16:21
@gadenbuie gadenbuie merged commit 00d956e into main Mar 19, 2024
@gadenbuie gadenbuie deleted the fix/input-dark-mode-style branch March 19, 2024 22:12
schloerke added a commit to joesho112358/py-shiny that referenced this pull request Mar 23, 2024
* main: (24 commits)
  chore: Disable broken test on CI; Fix broken docs entries (posit-dev#1241)
  Add back missing session methods (posit-dev#1239)
  Add kitchensink tests for valuebox (posit-dev#1229)
  For Express, emit message if need to upgrade rsconnect-python (posit-dev#1233)
  feat: Editable data frame (posit-dev#1198)
  Add tests for shiny_mode comment detection
  fix(input_dark_mode): Allow users to customize `style` attribute (posit-dev#1207)
  chore(get_current_session): Return default session if current is explicitly None (posit-dev#1086)
  chore: use dev htmtlools (posit-dev#1228)
  feat(layout_columns): Use `sm` as the default `col_widths` breakpoint (posit-dev#1222)
  feat: ensure min/max height args on `value_box()` and column layouts (posit-dev#1223)
  fix(static_assets): Need the directory containing the `app.py`, not the file itself (posit-dev#1219)
  Update bslib and shiny dependencies (posit-dev#1221)
  Update docstring and give hint for mypy
  Add support for Shiny Express in Quarto Dashboards, take 2 (posit-dev#1217)
  Force UTF-8 for Shiny Express on Windows (posit-dev#1203)
  Typing fixes
  feat(card): Report full screen state as a Shiny input (posit-dev#1215)
  Use TypeScript strict mode (posit-dev#1208)
  Update author email
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants