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(cdk): fix isFirefox check #158

Merged
merged 2 commits into from
Jan 27, 2021

Conversation

WynieCronje
Copy link
Contributor

@WynieCronje WynieCronje commented Jan 27, 2021

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows Conventional Commits
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

What is the current behavior?

Closes #156

What is the new behavior?

isFirefox check now works as expected

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@waterplea
Copy link
Collaborator

@WynieCronje thank you for the PR! Could you please go over other functions in that folder and add lowercasing too? Also, scope for commit message is the package. So in this case it would be fix(cdk): fix isFirefox check ;)

@WynieCronje WynieCronje changed the title fix(browser-detection): fix isFirefox check fix(cdk): fix isFirefox check Jan 27, 2021
@WynieCronje
Copy link
Contributor Author

WynieCronje commented Jan 27, 2021

@waterplea Do you prefer another commit?
Or should I squash the commits into one?
Or is this something that happens when PR get accepted?

@waterplea
Copy link
Collaborator

Don't worry, we squash when we merge PRs

@MarsiBarsi MarsiBarsi merged commit 8e9c7a4 into taiga-family:main Jan 27, 2021
fynnfeldpausch added a commit to fynnfeldpausch/taiga-ui that referenced this pull request Jan 28, 2021
* main:
  feat(demo): add page on custom dialogs (taiga-family#159)
  fix(cdk): fix isFirefox check (taiga-family#158)
  feat(demo): add guide on using different icon set (taiga-family#157)
  feat(demo): add `Wrapper` customization page (taiga-family#153)
  refactor(lang): translated russian text to english (taiga-family#155)
  docs(demo): fix typos
  feat(format): number formatting supports custom thousands separator (taiga-family#145)
  fix(core): fix new CSS vars for height name mismatch and add customization section to demo (taiga-family#149)
  chore(demo): fix typo
  chore(demo): try i18n
  chore(readme): update text
  chore(kit): `TabsWithMore` make class less generic to avoid collision (taiga-family#142)
  Revert "chore(package.json): try to change repo name for Github used by"
  chore(package.json): try to change repo name for Github used by

# Conflicts:
#	projects/addon-editor/components/edit-link/edit-link.template.html
#	projects/addon-editor/components/toolbar/toolbar.template.html
#	projects/demo/src/modules/markup/icons/icons.component.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] isFirefox util does not detect Firefox
3 participants