Skip to content

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Jan 8, 2026

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
Private
image 2026-01-08_11h41_25
Group
2026-01-08_11h25_19 2026-01-08_11h38_32
. 2026-01-08_11h38_54
. 2026-01-08_11h39_37
. 2026-01-08_11h40_24
Mobile -
image image

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@ShGKme
Copy link
Contributor

ShGKme commented Jan 8, 2026

Could you add screenshots for small screens?

@Antreesy
Copy link
Contributor Author

Antreesy commented Jan 8, 2026

Added for mobile resolution.

Need to address issue with border values slightly < isMobile (exists before PR):

UPD: moved to a separate issue

@Antreesy Antreesy marked this pull request as draft January 8, 2026 11:39
@ShGKme
Copy link
Contributor

ShGKme commented Jan 8, 2026

What about replacing "Meeting" (which can be even longer translated) with 📆 icon on mobile?
In the screenshot it takes ~50% of the space now.

@Antreesy
Copy link
Contributor Author

Antreesy commented Jan 8, 2026

What about replacing "Meeting" (which can be even longer translated) with 📆 icon on mobile?

isMobile is already a breaker, where we don't show any text, but icon:

https://github.com/nextcloud/spreed/pull/16686/changes#diff-6269ed7b241d5b26f61deb525df871280c5b333fe0b78a1f29241e4378631c1bR374-R382

Might be adjusted for border values

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Apart from the mobile one this looks good.

Would therefore propose to merge for now and fix mobile later which is not used in screenshots by marketing.

@Antreesy Antreesy marked this pull request as ready for review January 9, 2026 10:47
@Antreesy Antreesy merged commit 793bcf1 into main Jan 9, 2026
59 checks passed
@Antreesy Antreesy deleted the fix/16488/topbar-buttons branch January 9, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reshuffle upcoming meetings button Move Conversation actions to the right

4 participants