Skip to content

Add reset button#434

Merged
epasveer merged 2 commits into
epasveer:mainfrom
QuangNguyenMinh123:feature/Add_reset_button
Feb 21, 2026
Merged

Add reset button#434
epasveer merged 2 commits into
epasveer:mainfrom
QuangNguyenMinh123:feature/Add_reset_button

Conversation

@QuangNguyenMinh123
Copy link
Copy Markdown
Contributor

I want to add a reset button to Seer, but I don't know which icon I should use. Do you have any recommendations?

@QuangNguyenMinh123 QuangNguyenMinh123 marked this pull request as draft February 14, 2026 16:21
@epasveer
Copy link
Copy Markdown
Owner

Can you explain what "reset" does?

There is already a "terminate" button. And there is still a "restart" via ALT+R (but currently there's no button).

@QuangNguyenMinh123
Copy link
Copy Markdown
Contributor Author

Can you explain what "reset" does?

There is already a "terminate" button. And there is still a "restart" via ALT+R (but currently there's no button).

Assuming the program is being debugged, and I want to start all over again from the main function without terminate the current gdb session

@QuangNguyenMinh123 QuangNguyenMinh123 marked this pull request as ready for review February 19, 2026 04:58
@QuangNguyenMinh123
Copy link
Copy Markdown
Contributor Author

ALT+R

I didn't know that I could reset program this this short cut.
Have you ever considered setting it to be visible?
image

@epasveer
Copy link
Copy Markdown
Owner

Have you ever considered setting it to be visible?

I have. Here's a long thread from another user. #418

I'll review your PR. It should probably show the keys (like ALT+T)

@epasveer
Copy link
Copy Markdown
Owner

I made a small change to your PR.

I've added ALT+T to the Terminate button and ALT+R to the Restart button. There was also a small regression when starting Seer with no target specified. It always showed the Restart button even though no program was being debugged.

Give it a try and let me know if it works for you. Then I'll pull it.

Thanks.

@QuangNguyenMinh123
Copy link
Copy Markdown
Contributor Author

This looks good to me

@epasveer epasveer merged commit c6908db into epasveer:main Feb 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants