-
-
Notifications
You must be signed in to change notification settings - Fork 339
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
UpdateChangelogTask
broken
#5606
Comments
Indeed,
Agreed, it's a hacky workaround.
Could you please record a quick screencast? I have a CSS quickfix in mind, but I want to reproduce it locally first.
That's bad, but I think we need a webview for the login flow anyway, right? |
Thanks for the fix!
I can only scroll vertically a bit, a few pixels, maybe 30dp.
Currently, yes.
it is an option to change the login flow to using the external browser. The reason I changed it from initially being in the browser to later being in a WebView was primarily that it did not work with the LineageOS stock browser, IIRC. At least with LineageOS 18 (current is 24) it was still a problem: https://gitlab.com/search?search=ERR_UNKNOWN_URL_SCHEME+&nav_source=navbar&project_id=9202919&group_id=3936439&scope=issues |
Anyway, parsing the markdown in-app and then displaying this in a nice composable sounds like a nice little project we could do together for me. You learn a bit how to work with Compose and we eliminate the need for a WebView at that point in the app. Currently, the transformation into an HTML and displaying that HTML in a WebView is about 100 lines of code. An implementation in compose will probably not be much longer than that. My estimation would be around 200 lines for everything (i.e. including layout that would be in a layout XML in the classic Android View system approach). |
How to Reproduce
Run the
UpdateChangelogTask
. Notice that the html looks weird, i.e. starts like this:Subsequently the subsequent transformation doesn't work as expected: "Changelog" is not stripped and the last version for the changelog is not found.
Expected Behavior
Either make it work as before (no idea what changed that it suddenly does not work correctly anymore. Does the markdown parser not handle
\r\n
properly?) ...or
ditch this HTML stuff, because:
[<< v53.1] [v54.0] [v54.1 >>] or similar (a dropdown?) at the bottom to select for which version one wants to view the changelog. Would be more convenient too, I'd say
(ListItem being a Composable that would be defined in a few lines of code, basically a text to the end of a •)
@FloEdelmann
The text was updated successfully, but these errors were encountered: