Skip to content

Comments

feat: improve Appointment UI with dynamic importance badges and bold titles#1

Open
codeCraft-Ritik wants to merge 2 commits intoSayed94h:mainfrom
codeCraft-Ritik:main
Open

feat: improve Appointment UI with dynamic importance badges and bold titles#1
codeCraft-Ritik wants to merge 2 commits intoSayed94h:mainfrom
codeCraft-Ritik:main

Conversation

@codeCraft-Ritik
Copy link

Description: > I have improved the Appointment.js component to enhance visual hierarchy and urgency awareness.

Changes:

Dynamic Importance Badges: Replaced plain text with a color-coded system (Red for High, Gold for Medium, Green for Low).

Improved Scannability: Bolded appointment titles for better visual scanning.

Refactored Logic: Added a getImportanceClass helper function to keep the component code clean.

@Sayed94h
Copy link
Owner

Hi Ritik Kumar,

I appreciate your work on this project. You did a wonderful improvement. Excellent. I cannot merge it because as you know, this project is linked to a video tutorial and if it is not the same as I showed in the video, people will get confused. That's why it has to stay the same as the video tutorial.

Thank you for your understanding.

Best regards,
Aqyanoos Team

@codeCraft-Ritik
Copy link
Author

Thank you very much for reviewing my work and for your kind feedback. I truly appreciate your encouraging words.

I completely understand that the project needs to remain consistent with the video tutorial to avoid confusion for learners. Thank you for clarifying this, and I respect the decision.

I’m glad to have contributed and learned from this project. Looking forward to collaborating again in the future.

Best regards,

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.

3 participants