dashjoin / platform

Dashjoin Is an Open Source & Cloud Native Low Code Development and Integration Platform that helps teams deliver applications faster 🚀 Uses a data-driven approach inspired by Linked Data to make use of your existing assets
https://dashjoin.com
GNU Affero General Public License v3.0
89 stars 7 forks source link

Disable SideNav for user roles #356

Closed pgrill79 closed 1 week ago

pgrill79 commented 1 month ago

In Dashjoin 3.X it was possible to hide the Sidenave completely, now you can only set the properties:

"sidenav-open": "false" und "sidenav-width-px": "0"

but the Menu Button remains and only circles if you press it (and the text between open / close menu changes)

pgrill79 commented 1 week ago

sidenav-width-px: Width of the side navigation bar (changes applied after next login) - if the width is set to 0, the open / close button in the toolbar is also hidden

{ "homepage": "/page/master", "sidenav-open": "false", "sidenav-width-px": "0" }

I did this, but the Close Button is still there: image

aeberhart commented 1 week ago

You need to logout and back in. All UI config settings are transferred only when loggin in.

pgrill79 commented 1 week ago

I tried this (also cleared the cache), but no effect ... the Button on the top left is still there.

aeberhart commented 1 week ago

strange... just tested locally with different users and sidenav settings. did you also reload the browser to make sure the new single page javascript code is loaded?

pgrill79 commented 1 week ago

I just killed all chrome and edge processes and also cleard the cache, but nevertheless i see the button (in both browsers): image

These are the settings i have defined in the role of the user:

image

The version i use is: version 5.1.30-153bcf3-09d061b

aeberhart commented 1 week ago

ok - if the properties are used, the settings are stored as strings. Thus

    const sideNavWidth = profile.getVariable()['sidenav-width-px']
    const sx = sideNavWidth === 0 ? {

treats sidenavwidth as "0" != 0

aeberhart commented 1 week ago

fixed in 5.2 backend casts map values for boolean / integer profile settings