Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

System Messages .hasPendingMessages name might be incompatible with Mozilla's implementation #107

Open
mounirlamouri opened this issue Jun 27, 2013 · 2 comments

Comments

@mounirlamouri
Copy link
Contributor

Mozilla's implementation is using .mozHasPendingMessage() (without the 's'). If developers end up using this and have prefixed vs un-prefixed paths, it might break when the method get an 's' at the end.

@jmajnert
Copy link

jmajnert commented Jul 1, 2013

Both names seem fine. I guess if you guys already have external devs building apps using this method, then I don't see a reason we shouldn't remove the "s".

@anssiko
Copy link
Member

anssiko commented Jul 1, 2013

The singular form LGTM. We don't need another s-related issue cf. requestFullscreen() vs. requestFullScreen() :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants