Skip to content

Add missing actions to comment history panel #565

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LSUDOKO
Copy link

@LSUDOKO LSUDOKO commented May 16, 2025

This PR fixes #549

  1. Replace all HTML elements with Hugo shortcodes
  2. Document the three missing actions in the comment history panel:
  • Resolving individual comments
  • Muting notifications for specific comments
  • Deleting individual comments
  1. Fixed formatting by replacing HTML button elements with Markdown code blocks

The documentation now properly reflects all the recent advancements in the comment history panel functionality, including appropriate warnings about permanent actions like deletion and guidance on when to use each action.

Copy link

netlify bot commented May 16, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit bac4b08
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/68297d8faa27690008335f1d
😎 Deploy Preview https://deploy-preview-565--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.


### Commenting via hotkey

<button class="kbc-button kbc-button-xs">M</button> Add a comment into your design when you press "M" as the hotkey. After pressing the hotkey, a new comment will be displayed. You can move the new comment anywhere in your design.
`M` Add a comment into your design when you press "M" as the hotkey. After pressing the hotkey, a new comment will be displayed. You can move the new comment anywhere in your design.
Copy link
Member

Choose a reason for hiding this comment

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

Revert this change, please.

- <button class="kbc-button kbc-button-xs">Ctrl + C</button> / <button class="kbc-button kbc-button-xs">Ctrl + V</button>: Copy and paste
- <button class="kbc-button kbc-button-xs">Ctrl + Z</button>: Undo the action
- **Keyboard Shortcuts**: - `Ctrl + C` / `Ctrl + V`: Copy and paste
- `Ctrl + Z`: Undo the action
Copy link
Member

Choose a reason for hiding this comment

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

Revert these changes.

@leecalcote
Copy link
Member

Feedback given. I think it would be ideal if those actions that you describe are not done so strictly in context of the comment history panel, but are described at-large as each of those actions are available outside of the comment history panel as well.

@leecalcote leecalcote requested a review from zihanKuang May 25, 2025 23:55
@zihanKuang
Copy link
Contributor

Hi @LSUDOKO😊, are you still working on this, or have you encountered any problems?

@leecalcote
Copy link
Member

@LSUDOKO, are you still working on this PR?

Copy link

Checking in... it has been awhile since we've heard from you on this issue. Are you still working on it? Please let us know and please don't hesitate to contact a MeshMate or any other community member for assistance.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@leecalcote
Copy link
Member

@zihanKuang with no word from @LSUDOKO, please take over this PR and complete it or close it.

@LSUDOKO
Copy link
Author

LSUDOKO commented May 27, 2025

Apologies for the delay. I'm currently away from home for my college summer vacation and don't have access to my laptop. I’ll be able to address this issue after June 11, if that is okay with you

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.

[Docs] Add missing actions to comment history panel
3 participants