Skip to content

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Apr 15, 2024

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
imageimage imageimage

🏁 Checklist

  • 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
  • 🖥️ Tested with Desktop client or should not be risky for it
  • 🖌️ 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

@Antreesy Antreesy added 3. to review feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client labels Apr 15, 2024
@Antreesy Antreesy added this to the 💙 Next Major (30) milestone Apr 15, 2024
@Antreesy Antreesy requested a review from DorraJaouad April 15, 2024 09:24
@Antreesy Antreesy self-assigned this Apr 15, 2024
@nickvergessen
Copy link
Member

I would go with a https://nextcloud-vue-components.netlify.app/#/Components/NcCheckboxRadioSwitch Standard Switch to make it more clear, but then we don't have the space/place for the bell icon anymore

DorraJaouad
DorraJaouad previously approved these changes Apr 15, 2024
Copy link
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

Better approach 👍🏻. Until a new icon with small audio off on bottom right comes out
like this https://pictogrammers.com/library/mdi/icon/send-check/

@Antreesy
Copy link
Contributor Author

Antreesy commented Apr 15, 2024

NcCheckboxRadioSwitch

Not supported inside NcActions menu at the moment, so either this, or we might need to improvise

@DorraJaouad

This comment was marked as outdated.

@Antreesy Antreesy force-pushed the fix/12008/toggle-silent-chat branch from 2841a7f to 5811ad2 Compare April 15, 2024 09:56
@Antreesy

This comment was marked as outdated.

@DorraJaouad

This comment was marked as off-topic.

@nickvergessen
Copy link
Member

nickvergessen commented Apr 15, 2024

How about only doing these simple 3 lines (or on top of your changes, but basically without touching the send icon)

diff --git a/src/components/NewMessage/NewMessage.vue b/src/components/NewMessage/NewMessage.vue
index 4b1e45c25..b1523e694 100644
--- a/src/components/NewMessage/NewMessage.vue
+++ b/src/components/NewMessage/NewMessage.vue
@@ -141,6 +141,9 @@
                        <!-- Send buttons -->
                        <template v-else>
                                <NcActions v-if="!broadcast" :container="container" force-menu>
+                                       <template #icon>
+                                               <BellOffIcon v-if="silentChat" :size="16" />
+                                       </template>
                                        <NcActionButton close-after-click
                                                :name="silentSendLabel"
                                                @click="toggleSilentChat">
Normal Silent
Bildschirmfoto vom 2024-04-15 14-40-45 Bildschirmfoto vom 2024-04-15 14-40-49

@nickvergessen
Copy link
Member

Could then even keep the normal send-icon to reduce complexity

@Antreesy
Copy link
Contributor Author

How about only doing these simple 3 lines

If that wasn't a previous requirement to force three-dot there, that sounds like a better option for now

@nickvergessen
Copy link
Member

If that wasn't a previous requirement to force three-dot there

I think adding a custom icon was just added later to the component. So not a requirement but a past-restriction

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.

Suggestion by @nickvergessen sounds good to me, also with keeping the send icon normal (not outline) to reduce complexity as the bell-off icon is already indicating it.

@Antreesy Antreesy force-pushed the fix/12008/toggle-silent-chat branch from 5811ad2 to 2cd4e85 Compare April 15, 2024 14:44
@Antreesy
Copy link
Contributor Author

/backport to stable29

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

send without notification is not obvious enough

5 participants