-
Notifications
You must be signed in to change notification settings - Fork 277
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
enhc: Code Refactoring, Project Restructuring, and Unified Stylings #576
enhc: Code Refactoring, Project Restructuring, and Unified Stylings #576
Conversation
cddee04
to
ab813e8
Compare
ab813e8
to
6e9fb73
Compare
made actual fix removed module.css and unnecessary styles.js files
74b4c10
to
37aa205
Compare
added a new grid system without module css
37aa205
to
d7c458b
Compare
added proptypes
used Input component
da92d19
to
590f7a0
Compare
3df46a5
to
43e49e0
Compare
8b7436d
to
4c5b82f
Compare
…oxes and ImageGallery
removed random usage of backdrop and not following component architecture added attachmentpreview.styles.js
changed to styles.js
cf9286e
to
83db88e
Compare
9ea17d8
to
6c00087
Compare
c0b1a5f
to
a288820
Compare
Hey @sidmohanty11 , I've made some changes related to refactoring many components' code and unifying the stylings, and I've done a simple restructuring of the project, separating components and views. As far as I have tested, I'm seeing things working fine. However, since this PR has a lot of changes, there is a high chance of introducing bugs into the project, which I'll correct as soon as I notice them. Hence, I suggest creating a new branch 'v0.2' for the changes up to this point to preserve the project. Without that, it's unsafe to merge. Once you create a new branch, please review and merge it to 'develop' branch, and I'll fix any arising issues. Also, please don't squash and commit for this PR; otherwise, it will become difficult to go back to certain changes and drop or revert them. |
|
||
// Example Usage | ||
|
||
/* <Grid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
Brief Title
Acceptance Criteria Fulfillment
<div>
and<span>
to<Box>
.Fixes #575
Video/Screenshots
A video demonstrating that all major features are working properly after these changes.
2024-05-12.22-32-00.mp4