Skip to content

Refactored publish icon#1340

Merged
Rajesh-Nagarajan-11 merged 2 commits intolayer5io:masterfrom
kishore08-07:publish-icon
Feb 16, 2026
Merged

Refactored publish icon#1340
Rajesh-Nagarajan-11 merged 2 commits intolayer5io:masterfrom
kishore08-07:publish-icon

Conversation

@kishore08-07
Copy link
Contributor

Notes for Reviewers

  • This PR fixes [Bug]Public and Publish Icons Are Using the Same Asset #1336

  • This PR fixes a visual duplication issue where the “Public” and “Publish” icons were using the same icon asset.

  • The “Publish” icon has been updated to use the correct Publish icon to ensure clear visual distinction between the two actions and improve UX consistency.
    Signed commits

  • Yes, I signed my commits.

Signed-off-by: kishore08-07 <kishorebsm8@gmail.com>
Copilot AI review requested due to automatic review settings February 16, 2026 14:07
@kishore08-07 kishore08-07 changed the title refactored publish icon Refactored publish icon Feb 16, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses issue #1336 by updating the “Publish” icon so it no longer duplicates the “Public” icon asset, improving visual clarity between the two actions.

Changes:

  • Updated PublishIcon to use a different SVG path (intended to be the Material “Publish” icon).
  • Added a fill prop default for the PublishIcon path coloring.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kishore08-07 <kishorebsm8@gmail.com>
@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit 84564f8 into layer5io:master Feb 16, 2026
5 checks passed
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.

[Bug]Public and Publish Icons Are Using the Same Asset

2 participants