-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sample code for the article on building a contact book with Textual #573
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for providing the code, @lpozo! I have two comments. Please note that you may adjust the adjacent steps with the changes 🙂
data = [ | ||
("Linda", "111-2222-3333", "[email protected]"), | ||
("Joe", "111-2222-3333", "[email protected]"), | ||
("Lara", "111-2222-3333", "[email protected]"), | ||
("David", "111-2222-3333", "[email protected]"), | ||
("Jane", "111-2222-3333", "[email protected]"), | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When comparing my code with yours, I stumbled over this list. Maybe you can use the REPL example from the tutorial and add the content to the README file. Then, you can remove the list from here.
🔁 If you follow along, then you need to delete the list from the other steps, too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a leftover from my testing stage. Sorry.
("m", "toggle_dark", "Toggle dark mode"), | ||
("a", "add", "Add"), | ||
("d", "delete", "Delete"), | ||
("c", "clear", "Clear All"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Later in the tutorial you'll name the method .action_clear_all()
. You can either decide to rename the binding here, or rename the method later:
("c", "clear", "Clear All"), | |
("c", "clear_all", "Clear All"), |
Where to put new files:
my-awesome-article
How to merge your changes: