Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

feat(boards): add ability to update boards #45

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

benox3
Copy link
Member

@benox3 benox3 commented Jan 14, 2020

This adds a couple of changes.

  • Tiles will now have an editing state.
  • When creating a new board it will immediately route you to it.

};

const handleCancelClick = () => {
setName(name);
Copy link
Member

Choose a reason for hiding this comment

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

Question: what is difference between setName and setNextName?

@remarkablemark
Copy link
Member

Possible improvements:

  1. If user is editing card, pressing Enter will save the changes and not cause a page reload
  2. If user is editing a card, clicking outside the card will save the changes and exit the edit mode
  3. Clicking the card will allow the user to enter edit mode
  4. Do we want default placeholder text for every card? (I'm thinking no)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants