feat: marker for edited notes

[mateo-ivc]

Description

Adding a marker that indicates if a note has been edited or not

Changelog

dto/notes.go

• added field if a note has been edited

database/migrations

• added field to notes table, that stores if a note has been edited

database/notes.go

• updates the edited field in the database to true, since text has been changed

components/Note

• added marker beneath text
• added css where needed

Checklist

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• The light- and dark-theme are both supported and tested
• The design was implemented and is responsive for all devices and screen sizes
• The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes

[Schwehn42]

looking good all in all! 👍 I do have to backtrack about my opinion that having the edited class below doesn't matter though; having it inline would really look better since it doesn't impact the height. Only problem is that I have no idea how to achieve that 😅

[Resaki1]

looking good all in all! 👍 I do have to backtrack about my opinion that having the edited class below doesn't matter though; having it inline would really look better since it doesn't impact the height. Only problem is that I have no idea how to achieve that 😅

just my two cents, but I strongly believe this current solution to be better:

1. This way it is directly visible that the (edited) is not part of the note's text.
2. This is how other platforms solve it. Jakob's Law (hehe) states that users prefer our site to work like all other sites, and if you look at Whatsapp, Twitter etc., they solve it just like @mateo-ivc has solved it here.
3. I personally think increasing the height of the note slightly is less bad then cutting the text off earlier than necessary.

What do you think @Schwehn42?

[mateo-ivc]

do have to backtrack about my opinion that having the edited class below doesn't matter though; having it inline would really look better since it doesn't impact the height.

I talked with Selina about that. She said that it's okay this way. But if we find a solution I am happy to change it so that it is inline

[Schwehn42]

@Resaki1 @mateo-ivc good points, let's do it that way then 👍

waiting for #4180 first

[Schwehn42]

lookin' good 🙌

[github-actions]

The deployment to the dev cluster was successful. You can find the deployment here: https://4277.development.scrumlr.fra.ics.inovex.io
This deployment is only for testing purposes and will be deleted after 1 week.
To redeploy rerun the workflow.
DO NOT STORE IMPORTANT DATA ON THIS DEPLOYMENT

Deployed Images

• ghcr.io/inovex/scrumlr.io/scrumlr-frontend:sha-da0ad90

• ghcr.io/inovex/scrumlr.io/scrumlr-server:sha-da0ad90

[octomind-dev]

🐙 Octomind

Test Report: 13/14 successful.

description	status	details
About Section Visibility Test	Passed ✅	click
change avatar	Passed ✅	click
check Privacy Policy	Passed ✅	click
check terms & conditions	Passed ✅	click
close cookie banner - front page	Passed ✅	click
close cookie banner - sign-in	Passed ✅	click
create and delete board columns	Passed ✅	click
create_and_delete_notes_and_actions_v2	Failed ❌	click
create lean coffee board	Passed ✅	click
edit_notes_and_actions_v5	Passed ✅	click
get started	Passed ✅	click
share_session	Passed ✅	click
sign-in	Passed ✅	click
test all ways to open the setup flow	Passed ✅	click

commit sha: da0ad90