Skip to content

Conversation

@AndyScherzinger
Copy link
Member

Language mix is due to the lack of translations given it is a new string.

πŸ–ΌοΈ Screenshots

🏚️ Before 🏑 After
before after

…nd platform-aligned

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger added this to the 4.6.0 milestone Dec 18, 2025
@AndyScherzinger AndyScherzinger added design Design and UX/UI issues 3. to review labels Dec 18, 2025
@github-actions
Copy link
Contributor

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/2962.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

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

As discussed I’d prefer if it’s at the end of the list (and probably our "share link" should be a button as well?) but that’s more difficult and can be tackled separately.

This is a nice improvement. :)

@AndyScherzinger
Copy link
Member Author

As discussed in the chat let's leave it as is for the moment, the other items are part of the list itself, so it is technically doable to also add this to the recycler view I think but additional work larger than just the simple visual change of the PR.

Let's tackle it separately πŸ‘

@AndyScherzinger AndyScherzinger merged commit f471eb7 into main Dec 18, 2025
14 of 18 checks passed
@AndyScherzinger AndyScherzinger deleted the style/noid/externalSharing branch December 18, 2025 14:48
@AndyScherzinger
Copy link
Member Author

/backport to stable-4.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review design Design and UX/UI issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants