TotalRegistry icon indicating copy to clipboard operation
TotalRegistry copied to clipboard

Using Refresh resets Path

Open God-damnit-all opened this issue 1 year ago • 5 comments

___

The path gets reset after a refresh. Also, if the left sidebar is scrolled down, it will scroll back up to the top.

God-damnit-all avatar Jan 11 '24 02:01 God-damnit-all

Same issue here

Very disconcerting while doing exploration changing something, press F5 oops where was I ?!

https://github.com/zodiacon/TotalRegistry/assets/10621885/d493c828-7790-49bf-989d-5ea511124c0a

shodanx2 avatar Feb 13 '24 21:02 shodanx2

I'm aware, thank you for reporting! Right-click a key and select Refresh to refresh that node only (less disruptive).

zodiacon avatar Feb 13 '24 22:02 zodiacon

Hello,

It seems this issue is resolved in the new version

https://github.com/zodiacon/TotalRegistry/assets/10621885/45a1d904-8724-4cf0-a551-8dc2c3dc66ab

However I noticed, if you right click on the path and do refresh, it does not work (reliably ?)

Here is a demonstration

In this clip, I have created a key called Visible with a value of 0

Click refresh in the path context menu, does not reveal the new key.

However, pressing refresh on the toolbar works

https://github.com/zodiacon/TotalRegistry/assets/10621885/78fa7145-9db8-423c-a2ee-c11ed9b6a87e

shodanx2 avatar Feb 14 '24 05:02 shodanx2

@shodanx2 Well, first of all, let's get our terminology straight. I'm assuming we're going with the contemporary terms:

OFFICIAL CONTEMPORARY ᴋᴇʏ ɴᴏᴅᴇ sᴜʙᴋᴇʏ sᴜʙɴᴏᴅᴇ ᴠᴀʟᴜᴇ ᴋᴇʏ ᴛʏᴘᴇ ᴛʏᴘᴇ ᴅᴀᴛᴀ ᴠᴀʟᴜᴇ

I say this because I think that feature is intended to expose new subnodes. However I do agree it's unintuitive and it really should also refresh the node's keys.

@zodiacon Is this intended behavior? If so, will you consider revising it to also refresh the node's keys?

God-damnit-all avatar Feb 19 '24 20:02 God-damnit-all

Ok, if I understand correctly, node is like a folder, subnode like subfolder and key is like a file.

It might make sense to put key refresh in this menu

image

On the other hand, totalreg is so blazing fast that "refresh all" is the only action I really need

shodanx2 avatar Feb 19 '24 21:02 shodanx2