search

PikeNote / taskbar-groups-pike-beta

Lightweight application that lets users create and pin groups to the Windows taskbar och desktop
MIT License
126 stars 9 forks source link

When you add a new shortcut, previous last one doesn't get its arrows updated #37

Closed BanCrash closed 1 year ago

BanCrash commented 1 year ago

Hi, when you add a new shortcut to a group, the previous last one should have both arrows enabled because it's not the new last one item, but it doesn't happen that:

imagen

As you can see, last one and previous last one can't be moved below.

PikeNote commented 1 year ago

Fixed in https://github.com/PikeNote/taskbar-groups-pike-beta/commit/d4d92154af04d07659799e5299df39f1d2df7474 I changed some logic of how controls are added. In the past, for some reason, controls were added to the front rather than the end, which caused the UI controls for showing the shortcuts differing from the shortcut list stored in code. I have tested creating groups, editing them, and moving items around, and it seems alright atm. If you'd like to build and test the latest to ensure that, please do.

BanCrash commented 1 year ago

Fixed in d4d9215 I changed some logic of how controls are added. In the past, for some reason, controls were added to the front rather than the end, which caused the UI controls for showing the shortcuts differing from the shortcut list stored in code. I have tested creating groups, editing them, and moving items around, and it seems alright atm. If you'd like to build and test the latest to ensure that, please do.

Wow, that was fast, thanks!

I will test it later today or tomorrow.

BanCrash commented 1 year ago

Yes, this is fixed and also the perfomance is much better when moving the places, nice!