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

fix(toast): remove margin #13

Merged
merged 2 commits into from
Nov 16, 2023
Merged

fix(toast): remove margin #13

merged 2 commits into from
Nov 16, 2023

Conversation

braddialpad
Copy link
Contributor

Description

Removed unnecessary left margin on toast. It is not needed as gap is being used.

Pull Request Checklist

  • Ask the contributors if a similar effort is already in process or has been solved.
  • Review the contribution guidelines.
  • Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
  • Ensure all gulp scripts successfully compile.
  • Update, remove, or extend all affected documentation.
  • Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-13
🔨 If you experience an SSL issue then wait 2 minutes and try again.

@braddialpad
Copy link
Contributor Author

Since this causes a visual change, I've put a first-child selector instead, so it only applies the margin if a button actually exists inside the element.

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-13
🔨 If you experience an SSL issue then wait 2 minutes and try again.

Copy link
Contributor

@ninamarina ninamarina left a comment

Choose a reason for hiding this comment

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

Smart fix 👌

@braddialpad braddialpad merged commit a29bd03 into staging Nov 16, 2023
@braddialpad braddialpad deleted the fix/toast-remove-margin branch November 16, 2023 22:56
braddialpad pushed a commit that referenced this pull request Nov 17, 2023
## [8.21.4](dialtone/v8.21.3...dialtone/v8.21.4) (2023-11-17)

### Bug Fixes

* remove old icons from dialtone ([#11](#11)) ([0f6071f](0f6071f))
* **Toast:** remove margin ([#13](#13)) ([a29bd03](a29bd03))

### Documentation

* fix missing dependencies ([b9c41df](b9c41df))
* separate dialtone documentation site and library ([#6](#6)) ([c937293](c937293))
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