Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Misc Fixes for 2.0.3 #466

Merged
merged 8 commits into from
Mar 20, 2020
Merged

Misc Fixes for 2.0.3 #466

merged 8 commits into from
Mar 20, 2020

Conversation

alexpaxton
Copy link
Contributor

@alexpaxton alexpaxton commented Mar 20, 2020

Closes #456
Closes #457
Closes #462
Closes #467

Changes

  • Ensure Heading components that use the classnames from FunnelPage maintain proper margins
  • Fix appearance of notification dismiss buttons
  • Prevent overlay title from overflowing
  • Enable optional text wrapping in OverlayHeader
  • Fix appearance of toggle in visibility input

Screenshots

Screen Shot 2020-03-19 at 4 56 39 PM
Screen Shot 2020-03-19 at 4 56 52 PM
Screen Shot 2020-03-19 at 5 25 12 PM
vis-input
overlay-header-text-wrap-options

Checklist

Check all that apply

  • Updated documentation to reflect changes
  • Added entry to top of Changelog with link to PR (not issue)
  • Tests pass
  • Peer reviewed and approved
  • Signed CLA (if not already signed)

@alexpaxton alexpaxton requested a review from mavarius March 20, 2020 00:48
Copy link
Collaborator

@mavarius mavarius left a comment

Choose a reason for hiding this comment

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

Just one comment.

@@ -75,10 +76,16 @@
font-weight: $cf-overlay--title-weight;
color: $cf-overlay--title-color;
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should probably wrap the title instead of truncating it.

@alexpaxton alexpaxton merged commit 65ba005 into master Mar 20, 2020
@alexpaxton alexpaxton deleted the misc-fixes-2.0.3 branch March 20, 2020 19:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants