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

Next version bump (~v4.12.0) #738

Merged
merged 56 commits into from
Aug 25, 2021
Merged

Next version bump (~v4.12.0) #738

merged 56 commits into from
Aug 25, 2021

Conversation

teenwolfblitzer
Copy link
Member

@teenwolfblitzer teenwolfblitzer commented Aug 20, 2021

  1. feat(button group): add wrap option #711 (MED/HIGH) Adds option for button groups to wrap to multiple lines. Default behavior on mobile views has been changed to allow wrapping (other breakpoints are unaffected), so existing uses should be checked to ensure content still flows as expected. Examples include:
    • Website Pages and Website Landing Pages: Edit page view - Publish status
    • Coupons: Delete coupon confirmation modal
    • Offers: Offer export modal
    • Email campaigns: Broadcast send type selection
  2. Ensure expandable card is hidden by default #683 (LOW) Fixes a bug in Expandable Card to ensure it does not flash on page load before hiding. Note: Undo this patch after this version bump: https://github.com/Kajabi/kajabi-products/pull/20205
  3. Patch dropdown trigger leading #713 (LOW) Fix to line height in dropdown button trigger fixes tiny alignment issue.
  4. Input Group Message Bug #739 (LOW) Styling adjustment to prevent the button from stretching when input group has message/helper text. No impact is expected but a quick sanity check should be performed.
    • Settings/Account Details
  5. Sage dynamic select input - select2 #709 (LOW) Adds dynamic select input
  6. docs(master to main): update the repo's reference to main branch as main #732 (LOW) Docs only update; references to master have been removed
  7. [Sage-511] Sage 3 - Loader Update #737 (LOW) Updates progress bar height and add new "success" variant.
  8. Card row alignment schema #741 (LOW) Internal updates to schema for CardRow component. No impact on usages expected.
  9. Issue Template Update: Suggestion Template #730 (LOW) Docs only update.

@@ -31,7 +31,16 @@ def render
context.render(
partial: "sage_components/#{template_path}",
locals: { component: self }
).tap { cleanup_section_vars }
).tap do |html|
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change was reverted in main and will need to be reapplied here prior to the version bump merge.

@monicawheeler monicawheeler marked this pull request as ready for review August 25, 2021 15:57
@monicawheeler monicawheeler self-assigned this Aug 25, 2021
Copy link
Contributor

@anechol anechol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Monica Wheeler and others added 21 commits August 25, 2021 11:22
…nt contains pre html tags

We strip additional whitespace out of the partial's html to allow for cleaner rendering in the
browser. In cases where <pre></pre> html tags exist in the returned html, we do not strip any of the
whitespace in the partial. In the future we may want more specific string matching as <pre></pre>
tag usage grows.
…ontainer

margin removed on choice tabs within a button group container. the button group gap setting replaces
this need
replace docs description html with markdown
- remove `aria-expanded="false"` filter to aid default
- add `overflow: hidden` to aid hiding when `aria-hidden="true"` is not actvated.
Copy link
Contributor

@FuturaExtraBold FuturaExtraBold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@monicawheeler monicawheeler changed the title Next version bump (~v4.13.0) Next version bump (~v4.12.0) Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants