Theme Toggler Hydration Warning Handled

[walleeva2018]

This PR fixes the following Issue

What is Added

• In Navbar View Theme Switcher which should be a client-side component was rendering as a Server side component. Fixed that

[romangg]

Alternative is to use v-show instead of v-if. I think I would prefer that as more can be statically generated this way. What do you think?

[walleeva2018]

Alternative is to use v-show instead of v-if. I think I would prefer that as more can be statically generated this way. What do you think?

Yeah As Its navbar the load time will be faster. I will update that

[walleeva2018]

@romangg Let me know if there is any problem

[romangg]

With the v-show the change to ClientOnly is not needed anymore. So this change needs to be removed from the branch/PR.

[romangg]

The cleanest solution is to use interactive rebase for that: https://git-rebase.io/

[walleeva2018]

With the v-show the change to ClientOnly is not needed anymore. So this change needs to be removed from the branch/PR.

Yeah My bad. Is it okay now?

[romangg]

Thanks for the update. The problem is that this revision still contains changes that are as far as I see not needed anymore.