Sitelet https://web.archive.org/web/20220617133742/https://github.com/zulip/zulip/issues/22131
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add keyboard shortcut to "Drafts" link tooltip #22131

Open
alya opened this issue May 26, 2022 · 6 comments · May be fixed by #22149
Open

Add keyboard shortcut to "Drafts" link tooltip #22131

alya opened this issue May 26, 2022 · 6 comments · May be fixed by #22149

Comments

@alya
Copy link
Collaborator

@alya alya commented May 26, 2022

At present, while the left sidebar "Drafts" link tooltip includes the keyboard shortcut, the compose box "Drafts" link tooltip does not. We should fix this.

The compose box "Drafts" tooltip should thus be "Drafts (d)", with the same styling as the compose box x button tooltip:
Screen Shot 2022-05-26 at 4 20 30 PM

@zulipbot
Copy link
Member

@zulipbot zulipbot commented May 26, 2022

Hello @zulip/server-compose members, this issue was labeled with the "area: compose" label, so you may want to check it out!

@aparna-bhatt
Copy link
Collaborator

@aparna-bhatt aparna-bhatt commented May 27, 2022

Hey @alya I have solved this issue.But my previous PR Is almost ready to merge so can I contribute in this issue?
Pending PR #22129.
I am uploading the screenshot for you to see the changes.
Screenshot from 2022-05-27 15-46-59

@garg3133
Copy link
Member

@garg3133 garg3133 commented May 28, 2022

Sure @aparna-bhatt, you can go ahead and open a PR for this issue.

@aparna-bhatt
Copy link
Collaborator

@aparna-bhatt aparna-bhatt commented May 28, 2022

OK @garg3133 thanks!

@aparna-bhatt
Copy link
Collaborator

@aparna-bhatt aparna-bhatt commented May 28, 2022

@garg3133 I have made the PR can you review it?

@aparna-bhatt
Copy link
Collaborator

@aparna-bhatt aparna-bhatt commented Jun 3, 2022

Hey @alya can you review this PR.
Thanks!

aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 5, 2022
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 5, 2022
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 8, 2022
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 8, 2022
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 8, 2022
…#22131

compose: Add keyboard shortuct tooltip to Drafts link. Fixes zulip#22131.

compose: Add keyboard shortuct tooltip to Drafts link. Fixes zulip#22131.
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 8, 2022
aparna-bhatt pushed a commit to aparna-bhatt/zulip that referenced this issue Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants