Skip to content

Commit

Permalink
fix radio buttons and checkboxes rendering incorrectly; closes #55
Browse files Browse the repository at this point in the history
  • Loading branch information
daattali committed Jun 9, 2021
1 parent a15c95c commit 3bf43a7
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: shinyalert
Title: Easily Create Pretty Popup Messages (Modals) in 'Shiny'
Version: 2.0.0.9001
Version: 2.0.0.9002
Authors@R: c(
person("Dean", "Attali",
email = "daattali@gmail.com",
Expand Down
7 changes: 4 additions & 3 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,10 @@

- **BREAKING CHANGE** It's no longer required to call `useShinyalert()` in the UI. If you still want to explicitly pre-load the UI scripts, use `useShinyalert(force=TRUE)` (#47)
- **BREAKING CHANGE** The `rmd` parameter of `useShinyalert()` is no longer used. If you still want to call `useShinyalert()` (which you don't need to) then you have to remove the `rmd` argument
- Fixed bug: tall modals did not have a scrollbar and you couldn't see the entire content (#44)
- Fixed bug: long strings in the modal got cut off (#53)
- Fixed bug: vertical scrollbar incorrectly appeared when dialog has no buttons (#49)
- Fixed UI bug: tall modals did not have a scrollbar and you couldn't see the entire content (#44)
- Fixed UI bug: long strings in the modal got cut off (#53)
- Fixed UI bug: vertical scrollbar incorrectly appeared when dialog has no buttons (#49)
- Fixed UI bug: radio buttons and checkboxes did not render correctly (#55)
- New feature: added `session` parameter to allow advanced uses of shinyalert where the Shiny session isn't immediately accessible (thanks @galachad)

# shinyalert 2.0.0 (2020-09-11)
Expand Down
10 changes: 9 additions & 1 deletion inst/www/css/shinyalert.css
Original file line number Diff line number Diff line change
@@ -1,10 +1,18 @@
/* Ensure the modal has a vertical scrollbar if it's too tall */
.sweet-alert{
.sweet-alert {
max-height: calc(100% - 20px);
overflow-y: auto;
overflow-wrap: break-word;
}

/* Fix the look of radio buttons and checkboxes that are too big */
.sweet-alert input[type="radio"],
.sweet-alert input[type="checkbox"] {
height: auto;
margin-top: 4px;
width: auto;
}

/* Remove the error section as it's unused but causes a vertical scrollbar when there are no buttons */
.sweet-alert .sa-input-error {
display: none;
Expand Down

0 comments on commit 3bf43a7

Please sign in to comment.