Skip to content

Conversation

@perepujal
Copy link
Member

Also got trapped into a gradle update.
Java changes and browser interface are made mainly by Gemini.
This is my first serious approach to work with LLM, I hope I've managed to understand its code and to keep it small and clean.
Basically it creates a new activity for a webview and tweaks it to get the back button navigating the history like in a browser, instead of closing directly the activity.

The mkzipassets.sh changes are mine.

@automaton82 , could you test if it still compiles for you after the gradle updates? If needed I can try to downgrade them.
As for the docs browser, you will need an update of the assets, if you can't still run mkzipassets.sh, I've put a tar.gz with the updated assets at https://provant.freeddns.org/pere/public_html/Tux%20Paint/devel/20251201/assets.tar.gz about 115M

At the bottom of the config activity should show a "Browse" button that launches a webview for the docs, completely offline, the links that point outside the device will simply fail

Ouch! just noticed the button only shows in landscape mode, need to investigate why...

Thanks
Pere

Also got trapped into a gradle update.
Java changes and browser interface are made mainly by Gemini.
@automaton82
Copy link
Collaborator

Hi @perepujal I was able to download the assets, put them in the folder, and successfully build your branch without issue in Android Studio Otter | 2025.2.1 Patch 1.

As you noted, the button only shows up in landscape orientation, but I am able to open it successfully and see the documentation.

@perepujal
Copy link
Member Author

Thanks :)
Nice it compiles for you, I just found why the button didn't show up, already corrected in 3c605c5.

The next doubt is what to do with the links that fail because Tux Paint don't have internet connection, let them fail?
convert them to plain text? add the internet permission to Tux Paint?
In the last case, embedding the documentation is just redundant as one can just point the browser to the official doc site.
https://tuxpaint.org/docs

I guess I will let the links fail for now and see if there are complaints...

Thanks
Pere

Thanks
Pere

@automaton82
Copy link
Collaborator

Yea I think that's probably the best path for now.

@perepujal perepujal merged commit 8f3392e into tux4kids:master Dec 6, 2025
1 check passed
@perepujal
Copy link
Member Author

Merged, Thanks! :)

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.

2 participants