Skip to content
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

Added cross button in color gradient platte #90

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

DevanshPaliwal
Copy link
Contributor

@DevanshPaliwal DevanshPaliwal commented Jun 17, 2024

PR Description 📜

Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below -

Fixes issue #84 , added cross button in gradient palette, even if both colors are selected and we press this cross button, those colors will not be changed. There is the apply gradient button for that. This cross button correctly closes the gradient palette without any problem.


Mark the task you have completed ✅

  • I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
  • I have performed a self-review of my own code or work.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generates no new warnings.
  • I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
  • I have added screenshot for review.

## Add your screenshots(Optional) 📸

Screenshot is added in comments



Thank you soo much for contributing to our repository 💗

@DevanshPaliwal
Copy link
Contributor Author

Screenshot 2024-06-18 003533

@DevanshPaliwal
Copy link
Contributor Author

Here is the screenshot for the feature and proper working/testing has been performed by me before submitting PR.
Please label it under Gssoc and assign a level and merge into main branch @Durgesh4993

Copy link
Member

@Durgesh4993 Durgesh4993 left a comment

Choose a reason for hiding this comment

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

Ok

@Durgesh4993
Copy link
Member

@DevanshPaliwal make it transparesnt & add a demo vedio

@DevanshPaliwal
Copy link
Contributor Author

@Durgesh4993 where should i add the demo video?

@Durgesh4993
Copy link
Member

Yes

@DevanshPaliwal
Copy link
Contributor Author

Transparent button and demo video added

gradient.palette.demo.mp4

@DevanshPaliwal
Copy link
Contributor Author

@Durgesh4993 added changes.

@DevanshPaliwal
Copy link
Contributor Author

@Durgesh4993 please merge this PR under Gssoc24

@DevanshPaliwal
Copy link
Contributor Author

@Durgesh4993

Copy link
Member

@Durgesh4993 Durgesh4993 left a comment

Choose a reason for hiding this comment

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

Approved

@Durgesh4993 Durgesh4993 merged commit abe1d41 into ChromeGaming:main Jul 2, 2024
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.

2 participants