Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show location of entrances when adding/editing shops #5497

Open
rhhsm opened this issue Feb 22, 2024 · 6 comments
Open

Show location of entrances when adding/editing shops #5497

rhhsm opened this issue Feb 22, 2024 · 6 comments

Comments

@rhhsm
Copy link

rhhsm commented Feb 22, 2024

Use case
When editing shops and adding new ones in the right location, I think it is important to show the location of entrances. Often shops are located on the ground floor of apartment buildings that also have staircase entrances on that floor. To correctly add a new shop, it should be placed correctly relative to other shops (left/right of an existing one) but also correctly relative to entrances (left/right of one).

Proposed Solution
Show location of entrances in the shop overlay like they are already shown in the address overlay. I hope this is not too difficult and existing code can be copy-pasted. It would also be nice to have the entrances shown when answering a shop-related quest (name, hours, ...) like nearby shops are already shown, and like nearby entrances are shown in the entrance reference quest.

@FloEdelmann
Copy link
Member

Currently, things that are highlighted in an overlay are also editable, and this should not be possible for entrances in the shop overlay. Adding another set of highlighted elements that are just shown "for reference" break with that convention.

@rhhsm
Copy link
Author

rhhsm commented Feb 23, 2024

Currently, things that are highlighted in an overlay are also editable, and this should not be possible for entrances in the shop overlay. Adding another set of highlighted elements that are just shown "for reference" break with that convention.

I think that's a small price to pay for improved mapping accuracy. Strictly speaking, shops for shop-related quests and entrances shown at the entrance reference quest are not always editable (only when there's a quest for them). You'd have to switch to a suitable overlay if you want to edit them.

@westnordost
Copy link
Member

You expect to be able to map shops more precisely if the entrances are shown. Hm well, I never have had the impression that it would increase precision. After all, you can also see the building outlines, plus of course your own position.
Maybe for really long buildings...

Anyway, keep in mind that shop entrances may also be mapped and are not visually distinguished from "apartment/staircase entrances".

Maybe entrances could be shown as small dots instead of full icons, like in the address overlay.

@matkoniecz
Copy link
Member

After all, you can also see the building outlines, plus of course your own position.
Maybe for really long buildings...

https://www.openstreetmap.org/#map=19/50.07879/19.92230

screen01

@westnordost
Copy link
Member

Entrances with addresses are already shown, because addresses are shown.

@rhhsm
Copy link
Author

rhhsm commented Feb 24, 2024

You expect to be able to map shops more precisely if the entrances are shown. Hm well, I never have had the impression that it would increase precision. After all, you can also see the building outlines, plus of course your own position. Maybe for really long buildings...

The building that @matkoniecz showed above has entrance references and paths leading to the entrances mapped, so that helps to guess where they are. But for this building (screenshot below) it's hard to guess where they are, while I think it's important to be able to check that the order of shops and entrances is correct (i.e. from top to bottom it might be shop - shop - entrance - shop - entrance -etc.). Staircase entrances are nice "anchors" because they usually occur at regular distances: if an apartment building has 3 staircase entrances, they are very likely to occur at 1/6, 1/2 and 5/6 of the length of the building.

Screenshot (24 feb)

Anyway, keep in mind that shop entrances may also be mapped and are not visually distinguished from "apartment/staircase entrances".

It would work for me because very few shop entrances are mapped here, but it might be less helpful or even confusing for others.

Maybe entrances could be shown as small dots instead of full icons, like in the address overlay.

Yes, that's what I had in mind. And if they have to be editable, I wouldn't mind. Would be nice if in a suitable overlay (probably not the shop overlay) an entrance could be selected so that its type could be changed (for instance change entrance=yes to entrance=staircase, so that in the next step a staircase ref can be added with the entrance reference quest). But that's another issue...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants