Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add options to set position of notification (Top/Bottom)#29

Open
iNima wants to merge 1 commit into
MPGNotification:masterfrom
iNima:master
Open

Add options to set position of notification (Top/Bottom)#29
iNima wants to merge 1 commit into
MPGNotification:masterfrom
iNima:master

Conversation

@iNima

@iNima iNima commented Feb 22, 2015

Copy link
Copy Markdown

Hi @toblerpwn ,

I wanted to show the notification at bottom of screen.
Sometimes we face with a situation where it would be better to show notification at the bottom for example when It should remain on screen for a period of time. In some advance example we can show some count down or similar things as a notification. So it should not cover other UIs at the top of the screen.

I added an option to set notification position type.
Now developers can decide about showing notification at top or bottom.

Changing the position was not enough because all show and dismiss animations were implemented based on top position. So I changed all of them to be suitable and aligned based on position of notification.
All three animations now are shown and dismissed from/to direction that is appropriate for top/bottom notification.

I attached some screenshots, too.
Please try it and tell me your opinion about that. I tried to write the code in a pattern to be in sync with other parts of the source.

Regards,
Nima Azimi

pull01

pull02

pull03

pull04

…ss animation behaviors based on it. (Top/Bottom)
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.

1 participant