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

Closes #13 - adds top-center and bottom-center notifier position options #151

Merged

Conversation

christianklemp
Copy link

Also updates package.json to use needed grunt-rtlcss version (I wasn't able to get the .ajs- prefix addition working without explicitly downgrading the the changed-to version) and to use grunt-postcss with the autoprefixer plugin instead of the deprecated grunt-autoprefixer package that was no longer adding the vendor prefixes.

*Note: all whitespacing changes are due to my editor, Atom, performing clean up when I save files.

…fier position options. Updates package.json to use needed grunt-rtlcss version and to use grunt-postcss instead of the deprecated grunt-autoprefixer that no longer adds the vendor prefixes.
@MohammadYounes
Copy link
Owner

Thanks, I'll review this as soon as possible. However, this project has a commercial license and thus:

Do you consent that You provide Your Contributions on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied, including, without limitation, any warranties or conditions of TITLE, NON- INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A PARTICULAR PURPOSE ?

@christianklemp-imt
Copy link

👍 I do

@MohammadYounes
Copy link
Owner

Thank you, But I believe you should consent using the same account used to issue the PR.

@christianklemp
Copy link
Author

My bad! 👍 I do consent to the above statement.

@MohammadYounes
Copy link
Owner

Awesome... Thanks!

MohammadYounes added a commit that referenced this pull request Aug 4, 2017
Closes #13 - adds top-center and bottom-center notifier options
@MohammadYounes MohammadYounes merged commit 5975eeb into MohammadYounes:master Aug 4, 2017
@christianklemp christianklemp deleted the notifier-position branch August 6, 2017 20:17
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.

3 participants