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

Unable to delete items from results #16

Closed
codingo opened this issue Dec 5, 2017 · 6 comments · Fixed by #105
Closed

Unable to delete items from results #16

codingo opened this issue Dec 5, 2017 · 6 comments · Fixed by #105

Comments

@codingo
Copy link

codingo commented Dec 5, 2017

The ability to delete results from authorize in bulk would make cleaning through results much more efficient. Typically I will end up with some incorrect results when setting up my regular expressions that then get in the way when trying to sort by columns for bypassed successful logins later. It would be great to run some tests, delete the bad results and then start my testing.

@Quitten
Copy link
Owner

Quitten commented Dec 11, 2017

Is the 'Clear list' button is not enough? would you like to be able to remove specific fields? like via right click -> delete?
In addition, in case you would like to DIY, just send a pull request and let me know :)

@codingo
Copy link
Author

codingo commented Dec 12, 2017

Yes, thinking more right click - delete. It would make filtering results (by removing junk) much more efficient in large tests. A bit time pressured at the moment so not time for a PR but I can look into it in the new year if this is still outstanding.

@Quitten
Copy link
Owner

Quitten commented Jul 5, 2018

@codingo - I tried to implement this feature but I didnt have enough time to finish it, I currently pushed d93fd74 but there is a bug on line 1384. if you would like to help :)

@terminalJunki3
Copy link
Contributor

terminalJunki3 commented Jan 13, 2023

@codingo @Quitten Just added a PR but I think I have a working solution. Let me know if you see any bugs.

@terminalJunki3
Copy link
Contributor

terminalJunki3 commented Jan 13, 2023

@codingo @Quitten Was able to implement the multi-row select. You should now be able to select multiple rows by holding control and delete them. This should satisfy this request and this request should be able to be closed out. Code is in PR #105

@Quitten
Copy link
Owner

Quitten commented Jan 21, 2023

Reviewed and tested, merged.
Thank you very much for contributing!

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

Successfully merging a pull request may close this issue.

3 participants