Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

Adding Color picker for Events #8998

Merged
merged 7 commits into from
Sep 14, 2023
Merged

Conversation

AbhishekBhure
Copy link
Contributor

@AbhishekBhure AbhishekBhure commented Sep 8, 2023

Fixes Issue

fixes #8946

Changes proposed

-Declared a constant named inputClassName to change the classNames according to the type ="color" in the Input.js file
-added type="color" to the input field in [[data]].js

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Color-picker

Note to reviewers

@github-actions github-actions bot added the issue linked Pull Request has issue linked label Sep 8, 2023
styles/globals.css Outdated Show resolved Hide resolved
Co-authored-by: Tom Schmelzer <30869493+schmelto@users.noreply.github.com>
Copy link
Member

@eddiejaoude eddiejaoude left a comment

Choose a reason for hiding this comment

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

Looks good 👍 , just 1 inline comment

@AbhishekBhure
Copy link
Contributor Author

I have committed the changes, please review.

@AbhishekBhure
Copy link
Contributor Author

I have committed the changes please review

Copy link
Member

@eddiejaoude eddiejaoude left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@eddiejaoude eddiejaoude merged commit be95fad into EddieHubCommunity:main Sep 14, 2023
@AbhishekBhure
Copy link
Contributor Author

Thank you 👍

Great!!

theunhackable pushed a commit to theunhackable/theunhackable-LinkFree that referenced this pull request Sep 16, 2023
* added a scrollToTop button

* added ScrollToTop button and removed custom styles added to global.css

* changed styling according to the type of input and completed the assigned task

* Update styles/globals.css

Co-authored-by: Tom Schmelzer <30869493+schmelto@users.noreply.github.com>

* Update components/form/Input.js added switch statement

* Update pages/account/manage/event/[[...data]].js to set the color when editing

* Update components/form/Input.js

---------

Co-authored-by: Tom Schmelzer <30869493+schmelto@users.noreply.github.com>
Co-authored-by: Eddie Jaoude <eddie@jaoudestudios.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue linked Pull Request has issue linked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Color Picker for events
3 participants