Skip to content

Conversation

@KrishnaShuk
Copy link

@KrishnaShuk KrishnaShuk commented Mar 5, 2025

Issue(s)

#49

Acceptance Criteria fulfillment

  • A "Send" button is added to the contextual bar, positioned intuitively relative to the quick reply options.
  • The "Send" option is removed from the kebab menu.

Proposed changes (including videos or screenshots)

Screenshot from 2025-03-05 18-38-19
Screenshot from 2025-03-05 18-38-05
Screenshot from 2025-03-05 18-37-56

Video

Uploading Screencast from 2025-03-23 09-27-27.webm…

The current implementation of the Quick Reply app buries the "Send" option within the kebab menu of the contextual bar. While functional, this requires extra steps for users to send replies, slowing down the experience—especially for frequent users—and making it less intuitive than a direct action.

This PR proposes adding a dedicated "Send" button directly in the contextual bar, positioned alongside the quick reply options. This change allows users to send their selected or typed reply instantly without navigating the kebab menu. To maintain a clean UI, the "Send" option will be removed from the kebab menu.

@KrishnaShuk
Copy link
Author

@VipinDevelops i have added the separate send button in this PR.

@iam-vipin
Copy link
Collaborator

Share a video of complete working flow.

@KrishnaShuk
Copy link
Author

Hey @VipinDevelops , I have added the video showing the functionality of the send button.

@iam-vipin
Copy link
Collaborator

Can you show the working or edit and delete thing ?

@KrishnaShuk
Copy link
Author

@VipinDevelops I have updated the video. Now you can see all three functionalities - Send , Edit and Delete.

Copy link
Collaborator

@iam-vipin iam-vipin left a comment

Choose a reason for hiding this comment

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

There is alot of spaces changes in PR making it hard to review.

@@ -1,54 +1,54 @@
{
"id": "e664d2cb-7beb-413a-837a-80fd840c387b",
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we remove this space chagnes ?

@iam-vipin
Copy link
Collaborator

attached video is not playable

@KrishnaShuk
Copy link
Author

Okay i will make a new commit resolving the spaces issue and will attach a new video to it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants