Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Jun 20, 2025

  • Tests written, or not not needed

How to reproduce?

  1. Ensure there are enough files in the list to cover the full height of the device screen.
  2. Scroll down slightly.
  3. Observe that the Bottom Navigation View and the Plus (+) button disappear.
  4. Tap on any folder to navigate into it.
  5. Notice that within the folder, the Bottom Navigation View remains hidden, and only the Plus (+) button is visible.

Demo

abc.mp4

Signed-off-by: alperozturk <alper_ozturk@proton.me>
android:layout_gravity="center_horizontal"
android:gravity="center_horizontal"
android:orientation="vertical"
android:layout_marginBottom="@dimen/standard_double_margin"
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Footer and last item in the list must be visible, thus bottom margin needed.

Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
val binding = ListFooterBinding.inflate(inflater, parent, false)
OCFileListFooterViewHolder(binding)
}
else -> throw IllegalArgumentException("Unsupported view type: $viewType")
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value must be one of the following: VIEW_TYPE_HEADER, VIEW_TYPE_ITEM, or VIEW_TYPE_FOOTER. You can confirm this by reviewing the SectionedViewHolder implementation. Unfortunately, since the type is represented as an integer rather than an enum, a default else clause with an exception is required.

@github-actions
Copy link

test-Unit test failed, but no output was generated. Maybe a preliminary stage failed.

@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/15037.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

Copy link
Member

@tobiasKaminsky tobiasKaminsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please try to mimick the same behaviour as "+" button, instead of always showing bottom bar.

@alperozturk96
Copy link
Collaborator Author

#15063

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.

3 participants