TeamNewPipe / NewPipe

A libre lightweight streaming front-end for Android.
https://newpipe.net
GNU General Public License v3.0
31.7k stars 3.08k forks source link

Migrate empty_state_view xml/view to Jetpack Compose #11725

Closed Profpatsch closed 12 hours ago

Profpatsch commented 2 days ago

Followup rebase for https://github.com/TeamNewPipe/NewPipe/pull/11463 due to technical reasons.

Please see its description.

Profpatsch commented 2 days ago

A manual test clicking through the build on an empty newpipe seems that everything still works as expected.

Profpatsch commented 2 days ago

Some kaomoji that had different variants before seem to be the same now, maybe we want to keep that. Nevermind, that’s handled as well

Profpatsch commented 2 days ago

I’d merge soon, unless there are any vetoes

Stypox commented 1 day ago

Wait

Stypox commented 1 day ago

I think that the component's parameters are needlessly complicated. The only three items in the spec should be:

Stypox commented 1 day ago
sonarcloud[bot] commented 1 day ago

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Stypox commented 1 day ago

I pushed a few more commits, and now the code is better in my opinion. These things still need to be improved though (I don't know if it's better to do them now or in a future PR):

I tested all the screens above, in dark and light themes, and they work.

Profpatsch commented 1 day ago

Looks exceptionally good to me! (disclaimer: I didn’t test, since you already made screenshots)