Skip to content

Conversation

@wch
Copy link
Collaborator

@wch wch commented Oct 11, 2023

Previously, if two action buttons were next to each other, they looked like this:

image

After this change, they look like this:

image

With this change, the default is for input_action_button to call tags.button() with _add_ws=True, but it's done in such a way that the user can still set it to False.

App code:

from shiny import App, ui

app_ui = ui.page_fluid(
    ui.input_action_button("button1", "Button!"),
    ui.input_action_button("button2", "Button?")
)

app = App(app_ui, None)

@wch wch requested a review from schloerke October 11, 2023 17:57
@wch wch merged commit 0252141 into main Oct 27, 2023
@wch wch deleted the action-button-ws branch October 27, 2023 19:54
schloerke added a commit that referenced this pull request Nov 2, 2023
* main: (56 commits)
  Add actions step for nightly reporting -> Testrail (#774)
  Remove unused entry in manifest
  Bump version to 0.6.0.9000
  Use latest htmltools
  Bump version to v0.6.0; Rearrange news to have API changes last
  Tweaks from testing
  Fix default width/height with implicit plot output (#792)
  Update deps (#794)
  Remove deprecated 'name' parameter from `Outputs` (#791)
  api(ui): Drop `toggle_` methods. Consolidate update accordion methods. Stronger typing for `layout_sidebar(sidebar)` and `page_sidebar(sidebar)` (#788)
  bug(sidebar): Revert sidebar icon back to chevron (#789)
  For `input_action_button`, default to having whitespace around button (#758)
  Remove output from template app (#775)
  Add output_args and suspend_display decorators (#786)
  Update value_box; Update to bootstrap 5.3; Update htmldeps (#772)
  tests: add sidebar test (#787)
  Seaborn plots should fill their output_plot (#785)
  Kwargs to uvicorn run (#780)
  Add width and height arguments to `@render.plot` (#783)
  gitgnore dist/
  ...
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