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

feat(dialogs): extend MatDialogConfig to leverage all the dialog config (closes #1011) #1059

Merged
merged 5 commits into from
Jan 4, 2018

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Jan 2, 2018

Description

we are going away from hardcoding widths so this is technically a breaking change, but now you will be able to pass any widths and update the dialog configuration as needed.. also long words wrap properly

What's included?

  • Make sure long words wrap in the simple dialogs
  • Ability to pass any configuration property

Test Steps

  • npm run serve
  • Go to dialogs demo

General Tests for Every PR

  • npm run serve:prod still works.
  • npm run lint passes.
  • npm test passes and code coverage is not lower.
  • npm run build:release still works.

closes #1011

…igurations

we are going away from hardcoding widths so this is technically a breaking change, but now you will be able to pass any widths and update the dialog configuration as needed.. also long words wrap properly
@emoralesb05 emoralesb05 added this to the Release Candidate 2 milestone Jan 2, 2018
@emoralesb05 emoralesb05 changed the title feat(dialogs): extend MatDialogConfig to leverage all the dialog config feat(dialogs): extend MatDialogConfig to leverage all the dialog config (closes #1011) Jan 2, 2018
Copy link
Collaborator

@jeremysmartt jeremysmartt left a comment

Choose a reason for hiding this comment

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

Should this not be a breaking change? Maybe have the hardcoded value if you don't pass something in, which would make it not break, but if you do pass in something it will use that instead.

@jeremysmartt jeremysmartt self-assigned this Jan 2, 2018
@emoralesb05 emoralesb05 merged commit eb2ca43 into develop Jan 4, 2018
@emoralesb05 emoralesb05 deleted the feature/dialog-config branch January 4, 2018 00:22
kriswinbush pushed a commit to kriswinbush/covalent that referenced this pull request Feb 20, 2020
…ig (closes Teradata#1011) (Teradata#1059)

* feat(dialogs): extend MatDialogConfig to leverage all the dialog configurations

we are going away from hardcoding widths so this is technically a breaking change, but now you will be able to pass any widths and update the dialog configuration as needed.. also long words wrap properly

* feat(): add 400px width by default
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.

Simple dialogs do not support word-wrapping on strings that contain *very long* words
2 participants