Skip to content

feat: add UI for important/sensitive conversations#15115

Merged
Antreesy merged 5 commits intomainfrom
feat/noid/important-sensitive
May 21, 2025
Merged

feat: add UI for important/sensitive conversations#15115
Antreesy merged 5 commits intomainfrom
feat/noid/important-sensitive

Conversation

@Antreesy
Copy link
Copy Markdown
Contributor

@Antreesy Antreesy commented May 16, 2025

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
2025-05-16_17h52_35 2025-05-16_17h52_16
2025-05-16_18h06_59 image

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • 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

@ShGKme

This comment was marked as resolved.

Comment thread src/components/ConversationSettings/NotificationsSettings.vue
@Antreesy Antreesy requested a review from ShGKme May 17, 2025 10:11
@Antreesy
Copy link
Copy Markdown
Contributor Author

I pushed TS changes just before you approved 👀

@Antreesy Antreesy force-pushed the feat/noid/important-sensitive branch from 73f2336 to af8c08a Compare May 17, 2025 10:39
@Antreesy
Copy link
Copy Markdown
Contributor Author

/backport to stable31

Comment thread src/components/LeftSidebar/ConversationsList/Conversation.vue Outdated
Comment thread src/components/LeftSidebar/ConversationsList/Conversation.vue Outdated
Comment thread src/components/LeftSidebar/ConversationsList/Conversation.vue
Comment thread src/components/ConversationSettings/NotificationsSettings.vue Outdated
@Antreesy Antreesy changed the title feat: introduce API support for important/sensitive conversations feat: add UI for important/sensitive conversations May 19, 2025
Copy link
Copy Markdown
Member

@nimishavijay nimishavijay left a comment

Choose a reason for hiding this comment

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

Looks great! Could we make the subline a part of the toggle/checkbox in both the settings and the action menu?

@Antreesy
Copy link
Copy Markdown
Contributor Author

Updated:
image

Antreesy added 5 commits May 21, 2025 16:27
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the feat/noid/important-sensitive branch from 289c26f to e186f19 Compare May 21, 2025 14:37
@Antreesy Antreesy merged commit 741cd2a into main May 21, 2025
53 checks passed
@Antreesy Antreesy deleted the feat/noid/important-sensitive branch May 21, 2025 16:27
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.

Support "Important conversations" in the frontend

5 participants