Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

Added bunch of extra functionality #1

Merged
merged 21 commits into from
May 19, 2020
Merged

Added bunch of extra functionality #1

merged 21 commits into from
May 19, 2020

Conversation

kuukunen
Copy link

@kuukunen kuukunen commented May 8, 2020

Hi, I saw your fork of the project and started using it, but it was still lacking some features I wanted so I added them. I'll also send PR up to Grabacr07, but he seems to not be super active on it.

  • Rename desktops
  • Switch to specific desktop
  • Move window to specific desktop
  • Switch to previous desktop
  • Reset to default settings

Also bunch of tweaks.

kuukunen added 19 commits May 5, 2020 17:33
Combine shortcut settings menus to one.
Bump version mumber.
Doesn't use the same names as Windows, because the API is hidden and the
feature is currently only available in Insider Preview.
Also fix bunch of problems that came up.
…s to shortcut menu.

Reorganize settings panels.
Make default shortcuts more consistent.
Rename "index" to "number" for things like desktop "number" because they
start from 1 and it's confusing.
@ViRb3
Copy link
Owner

ViRb3 commented May 18, 2020

Hey there! I don't know how I have missed this, but I just saw it by pure accident. I will test, review, and hopefully merge ASAP. Thanks so much for the PR!

@ViRb3
Copy link
Owner

ViRb3 commented May 18, 2020

Here's a couple of issues I noticed during my initial test:

  • Rename window doesn't take focus. This means that the user has to click on the rename window in order to type in it, which I think is counter-productive and counter-intuitive.

  • Text not replaced in shortuct title:
    image

If you could kindly fix these bugs I can merge straight away. If you are busy I can do it as well tomorrow.

@kuukunen
Copy link
Author

Btw, there's a bit more info about possible issues and stuff on Grabacr07#46

Yea I noticed the text issue... but then promptly forgot about it. I've no idea how it was ever supposed to work. As a sidenote, it took me like an hour to figure out how to do properly... As mentioned in the above link, I'm not a C# developer and had no idea about WPF before modifying this.

And yea, the D1 etc comes straight from... I don't know where, but it's automatic.

As for the rename window focus thing, I totally agree, it should grab the focus. I tried to fix it multiple times until I got something that Works For Me™. It could be some sort of stupid race condition. However, since it works for me, I don't know how to fix it properly... I've no idea what would be the correct way of doing it either (see above), currently I just put it in the Activated event handler in RenameWindow.xaml.cs:79, if you want to take a look.

@ViRb3
Copy link
Owner

ViRb3 commented May 19, 2020

I will polish things up and hopefully merge this PR by the end of today :)

@ViRb3 ViRb3 linked an issue May 19, 2020 that may be closed by this pull request
@ViRb3 ViRb3 merged commit 298dade into ViRb3:develop May 19, 2020
@ViRb3
Copy link
Owner

ViRb3 commented May 19, 2020

I made a few small changes, but for your first C# WPF work I am amazed by the quality of this PR. Thank you so much on behalf of everyone who uses this tool!

@ViRb3
Copy link
Owner

ViRb3 commented May 19, 2020

Also good news @kuukunen, I managed to reliably fix rename window focus in 3c85113.

@kuukunen
Copy link
Author

Ah ok nice. I'm using it a lot right now myself, so might get annoyed by some more things in the future. We'll see.

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

Successfully merging this pull request may close these issues.

Inconsistent Notification screen preview
2 participants