-
Notifications
You must be signed in to change notification settings - Fork 2
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
Modernize script #14
Comments
@fenryxo I use the script, but now I do not have enough time to implement this amount of changes. I think that I will not be able to implement this changes untill September. |
Thanks for letting me know, @d-r-q. Could you at least implement "Semi-automatic changes" (that should be easy) and "Chromium-based backend and HTML5 playback without Flash plugin" (so the script won't stop working when WebKit backend is disabled) in the near future? |
Ok |
Blocked by tiliado/nuvolasdk#9. |
Hello @d-r-q. The freeze for Nuvola 4.12 is scheduled for July 1st (as per tiliado/nuvolaplayer#431). I hope we solve tiliado/nuvolasdk#9 and you manage to do "Semi-automatic changes", "Chromium-based backend and HTML5 playback without Flash plugin" and "Add web view snapshot" within that time period. If not, your script will be removed from the stable channel but may be introduced anytime later on (even in between Nuvola releases). |
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
…ut Flash plugin Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
…ut Flash plugin Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
…ut Flash plugin Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
…ut Flash plugin Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Signed-off-by: Aleksey Zhidkov <aleksey.zhidkov@gmail.com>
Blocked by:
tiliado/nuvolasdk#9Dear Maintainer,
There have been a lot of changes lately. I need you to review and modernize your script so that it keeps providing the best user experience possible. Eventually, I'd like to push some of them to Ubuntu software store as snap packages (tiliado/nuvolaplayer#273). If it increases the revenue of the project, I'd love to share it with the script maintainers (tiliado/nuvolaplayer#430).
I also need to know whether you still use this script and are willing to maintain it further. If you don't, I'll remove it from the stable repository until a new maintainer is found. No hard feelings though, you created & maintained it voluntarily and I'm glad for that.
Checklist
Use it to mark finished items.
.gitignore
is up-to-dateREADME.md
corresponds to the latest templatemetadata.in.json
uses two spaces for indentation and not trailing whitespaceintegrate.js
uses Standard JavaScript coding stylesrc/webview.png
) was added.Getting started
Since Nuvola migrated from DEB/RPM packages to cross-distribution Flatpak packages, the development also occurs inside Flatpak sandbox. All you need is to install Flatpak and Nuvola App Developer Kit (ADK), then you can start hacking. You don't have to worry about installing/building of dependencies, all you need is at your disposal. Learn more about Nuvola ADK.
The way the web app scripts are launched has changed as well. All you need is to launch Nuvola ADK,
cd
to your project directory and runnuvolaruntime -D
.Semi-automatic changes
nuvolasdk convert-project
. If there are any issues, please report them..gitignore
:nuvolasdk convert-project
should have fixed that.README.md
:nuvolasdk convert-project
will provide you with the new template (template--README.md
) and a difference between it and your README.md (template--README.md.diff
). Update your README.md to match the template - I will probably copy most of the template content and keep only the Copyright section from your current README.md.metadata.in.json
: Themetadata.in.json
now uses two spaces for indentation and no trailing whitespace.nuvolasdk convert-project
should have fixed that.integrate.js
: Nuvola adopted Standard JavaScript coding style. This allows us to automatically check the scripts for coding errors and mistakes.nuvolasdk convert-project
runsstandard --fix
, which tries to convert yourintegrate.js
.standard --fix
again.if-else
single lines.standard --fix
again and again...Chromium-based backend and HTML5 playback without Flash plugin
A long-time effort to get rid of Flash plugin may finally succeed. Nuvola had to switch a web rendering engine from WebKitGTK+ to Chromium Embedded Framework. Many scripts have already been ported, it's time to port the rest.
"requirements": "Chromium[65] Codec[MP3] Feature[MSE]"
tometadata.in.json
, run./configure
, and then launch your script withnuvolaruntime
. Remember that you need to run./configure
whenever you changesmetadata.in.json
."requirements": "Chromium[65] Codec[MP3] Feature[MSE] Feature[Flash]"
and test again.Update integration script
metadata.in.json
.Add new features
Several new features have been added to Nuvola. Look at them whether they apply to your web app.
Add web view snapshot
src/webview.png
../configure; make all
and look at the resulting screenshots in thescreenshots
directory.The text was updated successfully, but these errors were encountered: