Skip to content

Conversation

@dannyprikaz
Copy link
Member

Fixes #1701

What changes did you make and why did you make them ?

  • Replaced HTML div tag with MUI Box componenet
  • Replaced HTML button tag with MUI Button component
  • WHY: To modernize the editableMeeting.jsx component using MUI

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

1701 pre

Visuals after changes are applied

1701 post

@github-actions
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b dannyprikaz-issue1701-update-mui-editable-meeting development
git pull https://github.com/dannyprikaz/VRMS.git issue1701-update-mui-editable-meeting

Copy link
Member

@trillium trillium left a comment

Choose a reason for hiding this comment

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

Clean and easy changes, thank you10

Copy link
Member

@trillium trillium left a comment

Choose a reason for hiding this comment

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

Clean and easy changes, thank you!

@trillium trillium merged commit b74747c into hackforla:development Jan 27, 2025
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update HTML components to MUI: ./src/components/manageProjects/editableMeeting.js

2 participants