Skip to content

Leena Ibrahim#9

Open
Leena8051 wants to merge 2 commits intoGDSC-IAU:masterfrom
Leena8051:master
Open

Leena Ibrahim#9
Leena8051 wants to merge 2 commits intoGDSC-IAU:masterfrom
Leena8051:master

Conversation

@Leena8051
Copy link

No description provided.

@Leena8051 Leena8051 changed the title <commit-message> Leena Adel Jan 31, 2024
@Leena8051 Leena8051 changed the title Leena Adel Leena Ibrahim Jan 31, 2024
Copy link
Contributor

@RyamAlmalki RyamAlmalki left a comment

Choose a reason for hiding this comment

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

Overall, good job Leena!

myproject.dart Outdated
}

void Create() {
print("Enter note title: ");
Copy link
Contributor

Choose a reason for hiding this comment

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

check if the title already exists before creating a new note or if the title is empty when editing.

} while (optione != 5);
}

class NoteTakingApp {
Copy link
Contributor

Choose a reason for hiding this comment

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

separating the user interface logic from the application logic. This can make your code more modular and easier to maintain.

myproject.dart Outdated
}

void Edit() {
print("Enter a titel you want to edit: ");
Copy link
Contributor

Choose a reason for hiding this comment

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

Should have added an 'edit content' option for a better experience.

Copy link
Contributor

Choose a reason for hiding this comment

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

Additionally, check if the new title already exists before editing the note or if the new title is empty.

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.

2 participants