Skip to content
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

Added KiwiIRC as WebView component #20

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

Added KiwiIRC as WebView component #20

wants to merge 3 commits into from

Conversation

j-levy
Copy link
Contributor

@j-levy j-levy commented Sep 24, 2019

I thought it could be interesting to have access to R/a/dio IRC directly through the app. The website is a bit unpractical, and sometimes people just want to have a single app, not a second IRC client.
I added a 4th page with WebView displaying KiwiIRC-nextclient. (The "nextclient" version is more adapted for smartphones).
The theme used is "dark" to go with the rest of the app. There's probably room for improvement with tweaked theme or particular settings, but it works decently as it is currently.

App tested on various virtual devices (screen ranging from 800x480 to 1920x1080), on a Moto G5 Plus (Android 8.1, 5.2" screen 1920x1080), and on a Xiaomi Redmi 6A (Android 9, 5.5" 1440x720).

You can try the APK : https://shelter.mahoro-net.org/~yattoz/r-a-d.io-kiwiirc.apk

Added corresponding page, loads local HTML that contains an iframe with KiwiIRC. Used WebChromeClient to open links from chat.
KiwiIRC-nextclient is much more adapted to phones. I don't know how to roemove that big doggo though.
@resttime
Copy link
Contributor

resttime commented Nov 24, 2019

Can't test, but code looks fine in theory. I'd recommend heading to IRC and contact some staff for merging this. It's a new feature so the codebase could use a version bump along with an update on the play store by someone with the credentials.

@yattoz
Copy link

yattoz commented Nov 24, 2019

I wrote a complete new app (with this and much MUCH more). You can close this PR.
See #21 (comment)
(I'm the same guy as above)

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.

3 participants