Skip to content

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Aug 26, 2025

  • --color-success-text is the text color for --color-success
  • --color-text-success is the text color for normal backgrounds

πŸ–ΌοΈ Screenshots / Screencasts

🏚️ Before 🏑 After
Bildschirmfoto vom 2025-08-26 13-46-00 Bildschirmfoto vom 2025-08-26 13-46-10

🏁 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
  • ⛑️ Tests are included or not possible
  • πŸ“— User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@nickvergessen nickvergessen added this to the πŸͺΊ Next Beta (32) milestone Aug 26, 2025
@nickvergessen nickvergessen requested a review from Antreesy August 26, 2025 11:49
@nickvergessen nickvergessen self-assigned this Aug 26, 2025
@nickvergessen nickvergessen marked this pull request as ready for review August 26, 2025 11:49
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Maksim Sukharev <[email protected]>
@Antreesy Antreesy force-pushed the bugfix/noid/fix-used-color-variables branch from 830ecc5 to e3010a5 Compare August 26, 2025 12:21
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

Removed removed classes πŸ‘ŠπŸ½

- :class="{ 'success-button': virtualBackgroundAvailable === true, 'error-button': virtualBackgroundAvailable === false }"

@nickvergessen nickvergessen merged commit dbcbeda into main Aug 26, 2025
53 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/fix-used-color-variables branch August 26, 2025 14:17
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.

3 participants