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.

Sign up for GitHub

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

Jump to bottom

Ctrl+A functionality addition #24

Closed
Moodkiller opened this issue Mar 6, 2015 · 3 comments
Closed

Ctrl+A functionality addition #24

Moodkiller opened this issue Mar 6, 2015 · 3 comments

Comments

@Moodkiller
Copy link

It would be a great time saver when replacing the data/file location in the input field, to be able to use Ctrl+A to select all the text instead of having to right-click > select all and then subsiquently delete it.
@torque
Copy link
Contributor

torque commented Mar 6, 2015

Looks like the only thing stopping it from working was the abort button accelerator overriding the text box's default behavior. Will fix this after I make sure it works on Windows.

@Moodkiller
Copy link
Author

Sounds good and simple enough. Many thanks.

P.s. Sorry for the off topic question, but are you still available on IRC? torque on freenode and rizon? I have connected to both, it seems like the freenode one is more "active" than rizon

@torque
Copy link
Contributor

torque commented Mar 6, 2015

Hmm, disabling the button accelerator did not fix the problem on Windows like it did on OSX, so I'm going to have to chalk this up to being Aegisub's fault rather than mine. Will still shuffle the accelerators around, though.

And yes, I'm still on irc on both networks.

@torque torque closed this as completed Mar 6, 2015
torque added a commit that referenced this issue Mar 6, 2015
@torque
Aegisub-Motion: tweak accelerators a bit.
89f3050 
Was done in the hope of fixing #24, which it did... on OS X.
Unfortunately, ctrl-a to select all in the text box still does not work
on Windows, at least for me, so I can only assume it's somehow Aegisub's
fault. Since it fixes the issue on at least one platform and doesn't
cause different behavior on the main one, might as well commit the
change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@torque @Moodkiller