[BUG] Layer visibility btn

[h3llomoto]

If I change the state of the visibility btn - gjs will remove display style and then when I click "show layer" - it won't restore display style.
I think it's a bug. Gjs should save current display's options on layer visibility button click to restore it.

[artf]

I agree 👍

[h3llomoto]

OK guys I found a hotfix for this problem if you need:

        let itmView = editor.LayerManager.getAll();
        itmView.__proto__.toggleVisibility = function(e) {
            e && e.stopPropagation();
            const model = this.model;
            const style = model.getStyle();
            const hidden = style.display == 'none';

            if (hidden) {
                if (model.attributes.oldStyleDisplay) {
                    style.display = model.attributes.oldStyleDisplay;
                    delete model.attributes.oldStyleDisplay;
                } else {
                    delete style.display;
                }
            } else {
                model.attributes.oldStyleDisplay = style.display;
                style.display = 'none';
            }

            model.setStyle(style);
        }

[artf]

Thanks @h3llomoto the logic is totally correct, I'll make the same change but a bit more idiomatic