Together-100Devs / Together

Together is a group calendar application using the MERN stack intended to bring discord communities closer!
https://together.cyclic.app/
MIT License
167 stars 112 forks source link

Suggest creating "Description:" heading in Event details modal. #307

Open intelagense opened 1 year ago

intelagense commented 1 year ago

Please provide a summary of the feature

image

The event details modal inserts the word "Description: " as additional text before the actual description from the event creator. Semantically it would be better to include this in a heading tag, preferably above the text box area.

This would also improve the user experience when recreating events. Currently, when copying and pasting a previous event description in order to recreate a new event description the "Description:" can be copied and inadvertently duplicated as well.

Example:

"Description: Description: An event where people gather to do things."

Are there any potential issues you foresee with this feature

Aesthetics in regards to #17 👀

Is this feature related to any current or past issues? Please provide the

Additional context

Add any other context or screenshots about the feature request here.

Which Branch should this fix be Pulled into?

PR into [branchName]

If you're interested in working on this feature, please comment so you can be assigned

intelagense commented 1 year ago

link to some relevant code: https://github.com/Caleb-Cohen/Together/blob/343d7378306ad752cbada653b4de1f6d6dab829b/client/src/features/modal/EventModal.jsx#L64-L66

Caleb-Cohen commented 1 year ago

Just a heads up this has been solved in a future redesign in #285. Redesign will probably take some time so if someone wanted to take this on it will be available until #285 is pulled into development.

vguzman812 commented 1 year ago

Hi. This seems like an easy fix. I would like to be assigned this if possible. I already have a solved branch ready that will close this issue.

cblanken commented 1 year ago

This issue should be closed. It was resolved by PR #444