Skip to content
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

tagging an issue from the list should happen once the window is closed #595

Closed
karlcow opened this issue Mar 19, 2015 · 7 comments
Closed

Comments

@karlcow
Copy link
Member

karlcow commented Mar 19, 2015

  1. Go to an issue
  2. Select one or two tags

Actual: Tagging is happening right away as soon as we click the checkbox in the pop up window
Expected: Tagging should happen once leaving the pop up window.

Why: If we make mistakes in checking the boxes and changes while still in the pop up window, there is a dance of tagging happening in the history of the bug.

@miketaylr
Copy link
Member

I agree.

@miketaylr
Copy link
Member

Also a good opportunity to start on #272.

@miketaylr
Copy link
Member

(just testing how github behaves here)

@miketaylr
Copy link
Member

@karlcow just so I understand, we shouldn't update the UI with the labels that someone has picked until they Save/close the editor?

Right now we add/remove a label to the UI as soon as it is checked/unchecked, but don't send it to GitHub until the editor is closed.

I'm not attached to either behavior, so we can change it to only update UI when we send it to GitHub if you think that's a better model.

@karlcow
Copy link
Member Author

karlcow commented Apr 1, 2015

Ah! I guess I got confused by the UI of webcompat.com.
So I guess it's fine. Maybe yes not showing it in the UI until we close the UI.

@karlcow
Copy link
Member Author

karlcow commented Apr 1, 2015

Let's close it for now. Other priorities. And I will reopen that bug, when I'm in front of the same pattern which troubled me.

@karlcow karlcow closed this as completed Apr 1, 2015
@miketaylr
Copy link
Member

OK, thanks!

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

No branches or pull requests

2 participants