shape-builder icon indicating copy to clipboard operation
shape-builder copied to clipboard

Added the consistent box border on select to the output box

Open SplinterSword opened this issue 1 month ago • 11 comments

Current Behavior

Right now the box border box on select has a blue border which is not consistent with the theme of the website.

Image

Desired Behavior

The desired box border on select should be consistent with the theme of the website.

Implementation

Acceptance Tests

Mockups


Contributor Guides and Handbook

SplinterSword avatar Nov 03 '25 13:11 SplinterSword

@SplinterSword kindly assign me so that i can start my contribution

@Namanv0509 , could u please assign to me. I would like to work on this.

jagpreetrahi avatar Nov 03 '25 14:11 jagpreetrahi

@Samyukthaareddy-2410030191 sorry i am not a maintainer, I can't do assignments. You should ask @Namanv0509 or @M-DEV-1

SplinterSword avatar Nov 04 '25 04:11 SplinterSword

Hi @M-DEV-1, I hope you're doing well! I wanted to let you know that after the issue was initially assigned to me, I received the notification and started working on it right away. I’ve completed the implementation and have the solution ready. Would it be alright if I go ahead and open a PR for this? I believe it addresses the issue effectively. As for @Samyukthaareaddy, we can certainly find another good issue for them to work on as well. Thank you!

jagpreetrahi avatar Nov 04 '25 05:11 jagpreetrahi

sure

On Tue, Nov 4, 2025 at 11:23 AM Jagpreet Singh Rahi < @.***> wrote:

jagpreetrahi left a comment (meshery-extensions/shape-builder#77) https://github.com/meshery-extensions/shape-builder/issues/77#issuecomment-3484026655

Hi @M-DEV-1 https://github.com/M-DEV-1, I hope you're doing well! I wanted to let you know that after the issue was initially assigned to me, I received the notification and started working on it right away. I’ve completed the implementation and have the solution ready. Would it be alright if I go ahead and open a PR for this? I believe it addresses the issue effectively. As for @Samyukthaareaddy, we can certainly find another good issue for them to work on as well. Thank you!

— Reply to this email directly, view it on GitHub https://github.com/meshery-extensions/shape-builder/issues/77#issuecomment-3484026655, or unsubscribe https://github.com/notifications/unsubscribe-auth/BUUZM6D4JQGM6SSCRWGWHCT33A5OXAVCNFSM6AAAAACK7QVKT2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOBUGAZDMNRVGU . You are receiving this because you were assigned.Message ID: @.***>

--

www.klh.edu.in http://www.klh.edu.in

@jagpreetrahi if you have the solution ready then please go ahead and open a PR.

M-DEV-1 avatar Nov 04 '25 06:11 M-DEV-1

if you don't mind my friend wants to contribute as well her name is parnika can i ask her to message you in git so that you can add her as an assignee

On Tue, Nov 4, 2025 at 12:17 PM mahadevan @.***> wrote:

M-DEV-1 left a comment (meshery-extensions/shape-builder#77) https://github.com/meshery-extensions/shape-builder/issues/77#issuecomment-3484141560

@jagpreetrahi https://github.com/jagpreetrahi if you have the solution ready then please go ahead and open a PR.

— Reply to this email directly, view it on GitHub https://github.com/meshery-extensions/shape-builder/issues/77#issuecomment-3484141560, or unsubscribe https://github.com/notifications/unsubscribe-auth/BUUZM6BSKBLOHFJIHHHVIPT33BDZ7AVCNFSM6AAAAACK7QVKT2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIOBUGE2DCNJWGA . You are receiving this because you were assigned.Message ID: @.***>

--

www.klh.edu.in http://www.klh.edu.in

@M-DEV-1 , I have created a PR, now you can review it.

jagpreetrahi avatar Nov 04 '25 07:11 jagpreetrahi

Hi I’m Parnika and I would like to contribute to the issue @jagpreetrahi

darktadpoleofatinyswamp avatar Nov 04 '25 07:11 darktadpoleofatinyswamp

Hey @darktadpoleofatinyswamp , firstly thanking to your interest for contribute to this issue. But , I have already created a PR for this and asked for review by@M-DEV-1. So, I would like to suggest you to explore other open issue and ask to work on that. Thanks

jagpreetrahi avatar Nov 04 '25 07:11 jagpreetrahi

@Namanv0509 There is a conflicting issue here. I believe that earlier it was planned to remove the output box altogether and replace it with a "copy to clipboard" button. If that is still in the plan then this change might not be required.

M-DEV-1 avatar Nov 04 '25 10:11 M-DEV-1