Skip to content

Fix: Added missing images for Kanvas collaborate page#7695

Merged
saurabhraghuvanshii merged 7 commits intolayer5io:masterfrom
kanishksingh23:fix/kanvas-collaborate-page
May 5, 2026
Merged

Fix: Added missing images for Kanvas collaborate page#7695
saurabhraghuvanshii merged 7 commits intolayer5io:masterfrom
kanishksingh23:fix/kanvas-collaborate-page

Conversation

@kanishksingh23
Copy link
Copy Markdown
Contributor

Description
Adds the four images that were already imported in collaboration-feature-team.js
but were missing from the repo, causing the Kanvas collaborate page to be incomplete.

This PR fixes #6609

Changes made:

  • Added conversation.png
  • Added components.gif
  • Added activity-logs.png
  • Added undo-redo.png

Notes for Reviewers

  • These images were referenced in the existing collaboration-feature-team.js
    but never committed to the repo, which is why the page appeared incomplete.
  • This picks up the asset work from closed PR Fix: Kanvas Collaborate Page #6611.
  • Two additional items noticed for maintainer consideration (not in scope of this PR):
    • CollaborationFeatureCreate component exists but is commented out in index.js
    • CollaborationFeatureWork has world transition images commented out

Signed commits

  • Yes, I signed my commits.

Signed-off-by: kanishksingh23 <kanishksingh763@gmail.com>
@kanishksingh23 kanishksingh23 force-pushed the fix/kanvas-collaborate-page branch from eebdbaf to 22cd814 Compare May 4, 2026 06:36
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Preview deployment for PR #7695 removed.

This PR preview was automatically pruned because we keep only the 6 most recently updated previews on GitHub Pages to stay within deployment size limits.

If needed, push a new commit to this PR to generate a fresh preview.

@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@saurabhraghuvanshii, please review this.

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

@kanishksingh23 Please add all images and gif in /static/images and also update page.

Signed-off-by: kanishksingh23 <kanishksingh763@gmail.com>
@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@saurabhraghuvanshii, I have addressed your comment.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@kanishksingh23 it is good but can you please put these images into static/images/kanavs-collaborate ? that will be maintainable.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Signed-off-by: kanishksingh23 <kanishksingh763@gmail.com>
@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@saurabhraghuvanshii, gentle ping

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

@kanishksingh23 your changes are not visible https://layer5.io/pr-preview/pr-7695/cloud-native-management/kanvas/collaborate. Please take a look

@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@kanishksingh23 your changes are not visible https://layer5.io/pr-preview/pr-7695/cloud-native-management/kanvas/collaborate. Please take a look

I will look into this.

Signed-off-by: kanishksingh23 <kanishksingh763@gmail.com>
Signed-off-by: kanishksingh23 <kanishksingh763@gmail.com>
@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@saurabhraghuvanshii, the images are now loading correctly on the preview. Please take a look: https://layer5.io/pr-preview/pr-7695/cloud-native-management/kanvas/collaborate

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

Thanks @kanishksingh23 lgtm!!

@saurabhraghuvanshii saurabhraghuvanshii merged commit b929452 into layer5io:master May 5, 2026
4 checks passed
@kanishksingh23
Copy link
Copy Markdown
Contributor Author

Thank you @saurabhraghuvanshii, for the quick review.

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.

enhance collaborate page

2 participants