fix: allow UI settings to persist

[pbui]

UI settings were being overwritten on refresh. Changing this check appears to resolve this issue and allows for the UI settings to persist.

[vercel]

Someone is attempting to deploy a commit to a Personal Account owned by @Xyphyn on Vercel.

@Xyphyn first needs to authorize it.

[Xyphyn]

I'm unsure why I even have this mergeObjects function as a simple { ...obj1, ...obj2 } would probably work

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
photon	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 26, 2023 4:24pm

[Xyphyn]

The issue you're stating only seems to occur on some settings. I assume it's not saving actually, could you check local storage after you change a setting?

[pbui]

I did check the local storage and it was updating there. The issue appears to have been the mergeObjects function, which was overwriting the stored values.

[pbui]

Here is a screenshot from the [nu.lemdro.id] instance:

As you can see the revertColors and showInstance settings are updated in the local storage object.

[pbui]

I've updated the PR to remove the mergeObjects function and simply use the spread operator as you suggested.

I tested it briefly and this appears to work fine for allowing settings to persist.

[Xyphyn]

This works! Thanks for your contribution!