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

Confirm actions on index page #486

Merged
merged 2 commits into from May 23, 2013

Conversation

Projects
None yet
3 participants
@manuelmeurer
Contributor

manuelmeurer commented May 22, 2013

It has always bothered me that there is no "Seriously?" confirmation popup for the actions on the bottom of the problems index page (Merge, Unmerge, Resolve etc.)
I added the confirmation and refactored the buttons in the process.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 22, 2013

Coverage Status

Coverage remained the same when pulling d243465 on krautcomputing:confirm_actions_on_index_page into cab5c16 on errbit:master.

coveralls commented May 22, 2013

Coverage Status

Coverage remained the same when pulling d243465 on krautcomputing:confirm_actions_on_index_page into cab5c16 on errbit:master.

shingara added a commit that referenced this pull request May 23, 2013

@shingara shingara merged commit 541af98 into errbit:master May 23, 2013

1 check failed

default The Travis CI build failed
Details
@shingara

This comment has been minimized.

Show comment
Hide comment
@shingara

shingara May 23, 2013

Member

thanks

Member

shingara commented May 23, 2013

thanks

@manuelmeurer manuelmeurer deleted the krautcomputing:confirm_actions_on_index_page branch May 23, 2013

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