Skip to content

Rebase Re-Rethink fork onto upstream celzero/rethink-app#2813

Open
Mayanknishad9 wants to merge 1 commit into
celzero:mainfrom
Mayanknishad9:main
Open

Rebase Re-Rethink fork onto upstream celzero/rethink-app#2813
Mayanknishad9 wants to merge 1 commit into
celzero:mainfrom
Mayanknishad9:main

Conversation

@Mayanknishad9

Copy link
Copy Markdown

Summary

  • Rebases the Re-Rethink fork (Material 3 Expressive / Jetpack Compose UI rewrite and related changes) onto latest celzero/rethink-app main
  • Where histories diverged: UI/adapters/themes prefer the fork; core VPN, firewall, database, and wireguard services prefer upstream
  • Includes a large squashed commit bringing the fork work current with upstream

Test plan

  • Build and install the app (full/play flavors as applicable)
  • Smoke-test main screens (home, firewall, DNS, logs, settings)
  • Verify VPN connect/disconnect and basic firewall/DNS behavior still works
  • Review UI regressions vs upstream where Compose/Material 3 changes apply

Squashed Material 3 Expressive / Jetpack Compose UI rewrite and related
fork changes onto latest upstream main. UI/adapters/themes prefer the
fork; core VPN, firewall, database, and wireguard services prefer
upstream where histories diverged.
val scope = rememberCoroutineScope()
val explanation =
rememberDnsStatusExplanation(
key = "${endpoint.url}:${endpoint.blocklistCount}",
val isLastInGroup = nextInitial == null || currentInitial != nextInitial

if (isFirstInGroup) {
stickyHeader(key = "proxy_header_$currentInitial") {
}
}

item(key = "proxy_app_${item.uid}_${item.packageName}") {
}

if (index == 0 || isFirstInGroup) {
stickyHeader(key = "header_$currentInitial") {
}
}

item(key = "app_${item.uid}_${item.packageName}") {
verticalArrangement = Arrangement.spacedBy(2.dp)
) {
if (isLoading) {
item(key = "logs_loading") {
LogsLoadingState()
}
} else if (isEmpty) {
item(key = "logs_empty") {
LogsEmptyState()
}
} else if (hasLoadError) {
item(key = "logs_load_error") {
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