erisu commented on code in PR #1774: URL: https://github.com/apache/cordova-android/pull/1774#discussion_r1934933551
########## framework/src/org/apache/cordova/CordovaResourceApi.java: ########## @@ -45,21 +45,22 @@ Licensed to the Apache Software Foundation (ASF) under one /** * What this class provides: - * 1. Helpers for reading & writing to URLs. + * + * <p>1. Helpers for reading & writing to URLs. * - E.g. handles assets, resources, content providers, files, data URIs, http[s] - * - E.g. Can be used to query for mime-type & content length. + * - E.g. Can be used to query for mime-type & content length.</p> * - * 2. To allow plugins to redirect URLs (via remapUrl). + * <p>2. To allow plugins to redirect URLs (via remapUrl). * - All plugins should call remapUrl() on URLs they receive from JS *before* * passing the URL onto other utility functions in this class. - * - For an example usage of this, refer to the org.apache.cordova.file plugin. + * - For an example usage of this, refer to the org.apache.cordova.file plugin.</p> * - * Future Work: + * <p>Future Work: * - Consider using a Cursor to query content URLs for their size (like the file plugin does). * - Allow plugins to remapUri to "cdv-plugin://plugin-name/foo", which CordovaResourceApi * would then delegate to pluginManager.getPlugin(plugin-name).openForRead(url) * - Currently, plugins *can* do this by remapping to a data: URL, but it's inefficient - * for large payloads. + * for large payloads.</p> Review Comment: Applied `<ol>`, `<ul>`, and `<li>` tags. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org
