You think it's better to hide it in css ? Or totally wipe out in html ?

What param : new param 'visible'=> (true|false) (true by default) ?

If visible == false : don't display it.

Leave readonly like currently : if visible is false, readonly is false : you can add script (in future) to update it. But visible=>false,readonly=>true : didn't display + didn't allow update.

I think it would be safer to completely remove it from the HTML with a new parameter.

If you want it displayed but not editable (readonly), you can do something like I have done for "Use slider layout".

«If you want it displayed but not editable (readonly), » just add readonly => true attribute

It's done for this and really clean :) since https://github.com/LimeSurvey/LimeSurvey/commit/a875677a03f18b4cc505ef51379deaa29d2e2340#diff-51f80c6491f513287823eb97b46c89f2 it's cleaner in view
And since https://github.com/LimeSurvey/LimeSurvey/commit/de422484a4fddc0fd9603809965ec4950e0ffec3#diff-51f80c6491f513287823eb97b46c89f2 it disable update (and remove) in PHP too

Okay, I knew about <readonly_when_active>, but not simply <readonly>. I'll try that for "Use slider layout".

OK, testing currently some code about replace existing attribute …

There are a big issue : https://github.com/LimeSurvey/LimeSurvey/pull/1159 with included updated config.xml : ajax relaod didn't show location settings.
In fact : there are more issue : because current template are "auto_complete", then https://github.com/LimeSurvey/LimeSurvey/blob/378db43d7c58e4c9050cce6e91594d7331c7f1c7/application/controllers/admin/questions.php#L1680 unset it …

I see to fix this

        <!-- Hide map attribute -->
        <attribute>
            <name>location_city</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_state</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_postal</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_country</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>statistics_showmap</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_mapservice</name>
            <visible>0</visible>
            <readonly>1</readonly>
        </attribute>
        <attribute>
            <name>location_mapwidth</name>
            <visible>0</visible>
            <readonly>1</readonly>
        </attribute>
        <attribute>
            <name>location_mapheight</name>
            <visible>0</visible>
            <readonly>1</readonly>
        </attribute>
        <attribute>
            <name>location_nodefaultfromip</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_defaultcoordinates</name>
            <visible>0</visible>
        </attribute>
        <attribute>
            <name>location_mapzoom</name>
            <visible>0</visible>
        </attribute>

Fix committed to master_visibleOrNot_attribute branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=28409

@dominikvitt : can you confirm it's fixed in develop ?

Question : seems you version remove existing attribute from DB : did this attended behaviour ?

@tpartner : did you try with <types></type> ?

I test : it don't work when set template, but it‘s reaaly work after saving it

And this fix this issue too : https://bugs.limesurvey.org/view.php?id=14190

@DenisChenu: yes, it is fixed in develop.
It is intended behaviour.
I don't think you need to save unused attribute into database.

@dominikvitt, how is it fixed? Can you please point to some documentation?

@tpartner : did you try with <types></types> ? I test it (with autocomplete and location attributes): seems OK.

@dominikvitt : i think your current fix is more a kind of hack

Fixed in Version 3.1.7.3