hyperminimal icon indicating copy to clipboard operation
hyperminimal copied to clipboard

Published 20 hours ago • verified publisher icon jancborchardt

Adding instructions on Autohotkey script to move window in Windows OS

Open raulf2012 opened this issue 6 years ago • 6 comments

Following up on Issue #7 to add Autohotkey method of moving windows.

raulf2012 avatar May 20 '18 08:05 raulf2012

Thanks! :) Some improvements:

  • Could you cut it to 1 paragraph, with the link integrated?
  • The bold subheading is not needed as it repeats the first line of the following text.
  • We don't need the gif, it takes a bit of focus away from the screenshots.

Thank you!

jancborchardt avatar May 20 '18 17:05 jancborchardt

Hey Jan,

I've pushed the latest commit implementing your recommendations. Cheers.

raulf2012 avatar May 21 '18 05:05 raulf2012

Thanks! Some small changes, then it's good:

  • Can you directly link the text "use Autohotkey to execute a script"? Then it's really only one paragraph
  • […] with allows windows to be moved" should be "which", right?
  • "window of interest" can be cut to simply "window".
  • Could you squash your commits into a single commit for a cleaner history? :)

Thank you! :)

jancborchardt avatar May 21 '18 08:05 jancborchardt

@raulf2012 let me know if you need any help with what I suggested. :)

jancborchardt avatar May 24 '18 16:05 jancborchardt

Hey Jan, Sorry for the delay, I had never squashed commits before but I think I was finally able to figure it out. There should only be a single commit now. Can you confirm that everything is in order?

raulf2012 avatar May 24 '18 18:05 raulf2012

@raulf2012 no problem. :) So the text is good now, but if you look at the diff it seems for some reason the whole file is changed? https://github.com/jancborchardt/hyperminimal/pull/19/files

Maybe it’s easiest to just quickly start a new branch and paste in the changes. :) Then everything should be fine.

jancborchardt avatar May 28 '18 16:05 jancborchardt