Together icon indicating copy to clipboard operation
Together copied to clipboard

Published 20 hours ago • verified publisher icon Together-100Devs

Build reusable button component to close pop-up modals

Open intelagense opened this issue 2 years ago • 8 comments

Please provide a summary of the feature

Currently, the close buttons are centered at the top of the modals and says close Suggest turning the button into a reusable React component moving the close button to the right side of the modal and turning it into an X icon to match typical expected behavior.

Are there any potential issues you foresee with this feature

Needs to match the styling in #17

Is this feature related to any current or past issues? Please provide the

This was noticed with PR #333 into issue #303

Additional context

Add any other context or screenshots about the feature request here.

Which Branch should this fix be Pulled into?

PR into [development]

If you're interested in working on this feature, please comment so you can be assigned

intelagense avatar Feb 14 '23 20:02 intelagense

interested, please assign.

colinborows avatar Feb 14 '23 20:02 colinborows

Hey @colinborows I'm just checking in on this, thank ya!

Caleb-Cohen avatar Feb 28 '23 21:02 Caleb-Cohen

Hey! After a slight diversion, I am back on the React grind and working on this issue. I have been learning more about contexts and considering how best to implement this. I will reach out to the team with some questions this week.

colinborows avatar Feb 28 '23 23:02 colinborows

Update - reached a roadblock with this. Will be reaching out for help tomorrow.

colinborows avatar Mar 14 '23 21:03 colinborows

Update - reached a roadblock with this. Will be reaching out for help tomorrow.

Sounds good. Did you get the help you were looking for?

Caleb-Cohen avatar Mar 18 '23 15:03 Caleb-Cohen

@colinborows Caleb mentioned you might need some help with this issue. Feel free to reach out to me on discord @brotahman and we can tackle it 😀

luciousmc avatar Mar 21 '23 23:03 luciousmc

Update on this - @luciousmc has been kind enough to help me out with this over the past few days and figure out the best option. Now moving forward creating a reusable button that takes in an icon and a function, rather than simply a close button.

colinborows avatar Mar 28 '23 20:03 colinborows

Update on this - @luciousmc has been kind enough to help me out with this over the past few days and figure out the best option. Now moving forward creating a reusable button that takes in an icon and a function, rather than simply a close button.

Anything we can help ya with?

Caleb-Cohen avatar May 09 '23 01:05 Caleb-Cohen