Fix bottom sheet peek height to use resolveBottomSheetPeekHeight consistently

[Claude]

Addresses automated review feedback on bottom sheet safe-area handling. The peek height calculations were bypassing the base class's resolveBottomSheetPeekHeight() helper, causing inconsistent bottom inset handling between initial and dynamic updates.

Changes

• Remove manual navigationBarHeight from bottom margins - resolveBottomSheetPeekHeight() already handles safe-area insets, so manual addition caused double-counting
• Route dynamic peek height updates through resolveBottomSheetPeekHeight() - Ensures consistent safe-area behavior across gesture/button navigation devices

Before:

binding.opLl.updateLayoutParams<ViewGroup.MarginLayoutParams> {
    bottomMargin = requireContext().navigationBarHeight() + 24.dp
}
behavior?.peekHeight = binding.title.height + binding.scrollContent.height - ...

After:

binding.opLl.updateLayoutParams<ViewGroup.MarginLayoutParams> {
    bottomMargin = 24.dp
}
val contentHeight = binding.title.height + binding.scrollContent.height - ...
behavior?.peekHeight = requireContext().resolveBottomSheetPeekHeight(contentView, contentHeight)

Applies to both UserBottomSheetDialogFragment and GroupBottomSheetDialogFragment.

[Claude]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• dl.google.com
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED --add-opens=java.base/java.time=ALL-UNNAMED -XX:+UseParallelGC -Xmx10g -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en (dns block)
• https://api.github.com/graphql
  • Triggering command: /usr/bin/gh gh pr view --json number,title,body,reviews,comments /home/REDACTED/.nvm/bash_completion /home/REDACTED/.nvm/install.sh /home/REDACTED/.nvm/nvm-exec /home/REDACTED/.nvm/nvm.sh /home/REDACTED/.nvm/package.json /home/REDACTED/.nvm/rename_test.sh /home/REDACTED/.nvm/test (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)