ITU-BDSA2024-GROUP7 / Chirp

A project for a twitter clone called Chirp!
http://bdsagroup07chirprazor.azurewebsites.net/
MIT License
1 stars 0 forks source link

Have you ever seen an awesome Cheep That you wanted to Answer directly? Introducing Comments! #205

Closed DragonFlyersx closed 2 days ago

DragonFlyersx commented 4 days ago

Have you ever seen an awesome Cheep That you wanted to Answer directly? Introducing Comments!

With Comments you now have the ability to comment on the every cheep

This means that you can keep a conversation going about a specific topic without blowing up your timeline

The Comments will be located with " View Comments" These will displayed underneath every single cheep on the site.

When pressing view comments you will be taken to a new page that handles and shows all the comments

Skærmbillede 2024-11-30 140150

Here you will be able to post your new comments and add them to the thread

If you want to remove a message you are able to do so by clicking delete

For the people Behind the scenes

DragonFlyersx commented 3 days ago

Great implementation with comments!

I think the decision with creating a dedicated page is much better than if the comments where nested underneath the Cheep.

However if im not wrong (looking from phone), the comment page doesnt show if there is an image in the cheep. You can talk to @Nikolaj787 about how to implement that or me in the evening tomorrow. You can also just look at the implementation in public timeline.

Furthermore I kind of miss a indicator whether or not a Cheep has any comments. Maybe it can be done like "Comments (7)". Maybe its just me, but it could be a QoL improvement, and i dont think its a hard implementation. (You can just use the same query for fetching comment and add a .Count to get the amount of comments.

Otherwise im really impressed, and now stop disliking my comments #StopMobning xD

I will try to look into it and see if it possible to add Images might be cool to add aswell I will have a look but I don't want to create a new migration since that might be nessecary