notifyjs icon indicating copy to clipboard operation
notifyjs copied to clipboard

Center middle not recognized

Open scabros opened this issue 11 years ago • 7 comments

hi,

the middle option in position is not recognized. logs this to the console:

Uncaught Must be one of [t,b,l,r] notify.min.js:5
t.getPosition notify.min.js:5
t.setGlobalPosition notify.min.js:5
t.run notify.min.js:5
t notify.min.js:5
n.(anonymous function) notify.min.js:5
checkData main.php:56
llamarMozo main.php:63
jQuery.event.dispatch jquery-2.0.3.js:4676
elemData.handle jquery-2.0.3.js:4360

scabros avatar Jan 20 '14 13:01 scabros

I think you need to horizontally align before you vertical align, try left middle

jpillora avatar Jan 20 '14 13:01 jpillora

hi, thanks for your quick response.

now "left middle" is working fine.

but "center middle" show the same error above...

scabros avatar Jan 20 '14 14:01 scabros

Ah yes now I remember, center middle isn't supported, needs a PR :)

On Tue, Jan 21, 2014 at 1:04 AM, gitscattini [email protected]:

hi, thanks for your quick response.

now "left middle" is working fine.

but "center middle" show the same error above...

— Reply to this email directly or view it on GitHubhttps://github.com/jpillora/notifyjs/issues/9#issuecomment-32761461 .

jpillora avatar Jan 20 '14 14:01 jpillora

what is a PR?

thanks anyway!

scabros avatar Jan 20 '14 14:01 scabros

A PR is a Pull Request, because it's open source, you can add in the features you like and then send your changes back to me to be merged in with this version.

jpillora avatar Jan 21 '14 01:01 jpillora

any update ?

Jeje2201 avatar Feb 22 '21 19:02 Jeje2201

Not solved yet?

Vitalicus avatar Jun 20 '22 14:06 Vitalicus