<html><head></head><body><div class="ydp8ff9024byahoo-style-wrap" style="font-family: times new roman, new york, times, serif; font-size: 16px;"><div><div dir="ltr" data-setdir="false">But, as I am sure has been said by someone else recently:</div><div dir="ltr" data-setdir="false"><br></div><div dir="ltr" data-setdir="false">a.  A shop is still a shop, even when it is closed.</div><div dir="ltr" data-setdir="false">b.  A permanently closed shop that still has the signage / branding all over it is still a useful landmark.</div><div dir="ltr" data-setdir="false"><br></div><div dir="ltr" data-setdir="false">So, I think I would favour disused:shop=*  That way you know from the outset that you won't be able to buy anything there atm, but all the other details, like name= can be retained.</div><div><br></div><div class="ydp8ff9024bsignature"><div style="font-family:new times, serif;font-size:16px;"><div>Regards,</div><div dir="ltr">Peter</div></div></div></div><div><br></div><div><br></div>
        
        </div><div id="ydpa8c4b3c4yahoo_quoted_0190175836" class="ydpa8c4b3c4yahoo_quoted">
            <div style="font-family:'Helvetica Neue', Helvetica, Arial, sans-serif;font-size:13px;color:#26282a;">
                
                <div>
                    On Wednesday, 25 September 2019, 14:03:21 BST, Andy Allan <gravitystorm@gmail.com> wrote:
                </div>
                <div><br></div>
                <div><br></div>
                <div><div dir="ltr">On Wed, 25 Sep 2019 at 14:51, Andy Mabbett <<a shape="rect" href="mailto:andy@pigsonthewing.org.uk" rel="nofollow" target="_blank">andy@pigsonthewing.org.uk</a>> wrote:<br clear="none">><br clear="none">> On Wed, 25 Sep 2019 at 13:38, Dave F via Talk-GB<br clear="none">> <<a shape="rect" href="mailto:talk-gb@openstreetmap.org" rel="nofollow" target="_blank">talk-gb@openstreetmap.org</a>> wrote:<br clear="none">> ><br clear="none">> > Because shop=* indicates it is still open for business.<br clear="none">><br clear="none">> If it does not do so in "shop=vacant" then it does not do so in<br clear="none">> something like "opening_hours = none".<br clear="none"><br clear="none">What you're proposing here is "oh no it isn't" tagging, which is<br clear="none">generally frowned on.<br clear="none"><br clear="none">amenity = pub <- "yay, this is a pub"<br clear="none">opening_hours = none <- "oh no it isn't"<br clear="none"><br clear="none">Any map, app, geocoder etc that doesn't parse the additional tags is<br clear="none">therefore mislead. It's much more preferable to avoid confusion by<br clear="none">changing the main tag, in this case changing the "shop" tag to<br clear="none">something else (like vacant) or removing it entirely by converting it<br clear="none">into disused:shop tag.<br clear="none"><br clear="none">Thanks,<div class="ydpa8c4b3c4yqt0114540614" id="ydpa8c4b3c4yqtfd54520"><br clear="none">Andy<br clear="none"><br clear="none">_______________________________________________<br clear="none">Talk-GB mailing list<br clear="none"><a shape="rect" href="mailto:Talk-GB@openstreetmap.org" rel="nofollow" target="_blank">Talk-GB@openstreetmap.org</a><br clear="none"><a shape="rect" href="https://lists.openstreetmap.org/listinfo/talk-gb" rel="nofollow" target="_blank" fg_scanned="1">https://lists.openstreetmap.org/listinfo/talk-gb</a><br clear="none"></div></div></div>
            </div>
        </div></body></html>