Skip to content

Plugin Audit proposal #58

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

stevengill
Copy link
Contributor

Realized i accidentally closed the plugin audit PR.

#28

* Open Pull Requests: 3
* Spec: http://www.w3.org/TR/battery-status/
* android 5 has the ability to grab battery info in uiwebview built in
* isPlugged is useful but not being used much
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can you tell that isPlugged is not being used much?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Question. I don't think we have any stats.

* Open Issues: 71
* Open Pull Requests: 10
* Spec: http://www.w3.org/TR/contacts-api/ (discontinued)
* API needs to be redone. Currently brittle and many platform specific quirks
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide an example of brittleness? I'm curious what the perceived failings are.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it was more around the fact that the tests pass/fail randomly it seems.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we properly reflect that in the audit document, then?

* Too many quirks between platforms
* Maybe implement web audio api spec instead as a plugin
* web audio supported in iOS 8.4+ and Android 5+
* create web audio plugin for Android 4 to 4.4
Copy link

@filmaj filmaj Dec 14, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the latest cordova-android release dropped support for Android 4, no?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4.1 and up are still supported, it was just 4.0.x that was dropped

* looks ugly on ios
* currently not sync which is how default alerts work
* window.alert still available but says index.html
* Recommendation: Deprecate! Encourage use of HTML/CSS/JS divs instead.
Copy link

@brody4hire brody4hire Dec 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-1 please do not deprecate, reasons given in comments below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Appreciate you providing feedback :)

@brody4hire
Copy link

I really hope the dialogs plugin does not go away for the following reasons:

  • portable alert mechanism which can be handy for test and demo programs
  • easy plugin for newbies to get started with

For example my https://github.com/brodybits/Cordova-sqlite-bootstrap-test sample uses the dialogs plugin.

@filmaj
Copy link

filmaj commented Jan 20, 2017

May I suggest we pull download numbers once more, and compare against the 2015 numbers? Might provide a bit more insight.

stevengill and others added 8 commits February 1, 2017 13:11
* Can we treat some urls consistency across all platforms? Possibly use Flex Air package for inspiration.
* ex: app:, temp:,
* cdvfile wasn’t consistent enough across platforms
* Recommendation: Keep. Work on creating more consistency across platforms. Especially with cdvfile. Create phonegap-plugin-file with improved api.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worth noting that our current implementation is a superset of the actual W3C spec: https://www.w3.org/TR/FileAPI

Things like FileEntry and similar APIs look to be discontinued now.

Our implementation is out of date.

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.

5 participants