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

'both' to separate shows UI with three states #48

Open
tordans opened this issue Sep 17, 2021 · 2 comments
Open

'both' to separate shows UI with three states #48

tordans opened this issue Sep 17, 2021 · 2 comments

Comments

@tordans
Copy link
Collaborator

tordans commented Sep 17, 2021

image

Steps to reproduce

  1. https://zlant.github.io/parking-lanes/#17/52.49471/13.44110
  2. click the segment from the screenshot
    => it shows both, but the north is no_stopping now
  3. remove "both"
    => Issue 1: I expected the values of ":both" to be copied to :left, :right, since one of those will most likely stay the same. I wanted to change just the other
  4. re-add the values for the south that where on :both before
  5. now click on the segment to the left; it has a :both tagging as well
  6. now click back to the initial segment
    => Issue 2: It shows the state from the screenshot. But the first section with "both" should not be there
@zlant
Copy link
Owner

zlant commented Sep 17, 2021

  1. This is by design. If specified :both and :right/:left tags, both sections will be shown.

@tordans
Copy link
Collaborator Author

tordans commented Sep 17, 2021

But I removed the both in step 3 which did remove it from the UI so I assumed it also removed it from the local data.

Then IMO the issue is, that when removing the "both" checkmark, the local data should be copied to left, right (see issue 1) and removed from both.

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

No branches or pull requests

2 participants