Create the hide note layer button when that layer is active#6891
Closed
Robot8A wants to merge 4 commits intoopenstreetmap:masterfrom
Closed
Create the hide note layer button when that layer is active#6891Robot8A wants to merge 4 commits intoopenstreetmap:masterfrom
Robot8A wants to merge 4 commits intoopenstreetmap:masterfrom
Conversation
Collaborator
|
I don't think this is a good approach, for a few reasons:
So overall I don't think this is the answer. |
Member
|
This is better than #6890 but I agree with @gravitystorm that it's not really very nice. I suspect the correct solution is along the lines of that originally proposed in #867 of having the note layer turn off once the panel is closed - possibly the new note alone should be left to address my concerns in #867 (comment)? |
Collaborator
|
@Robot8A Thanks for your suggestion, but based on the comments above we're going to decline this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Would solve #867 by adding a new button to hide the note layer just below the add note button. This button would only appear when the note layer is active. A user who activated the layer by clicking the add note button would expect a button to hide it in the vicinity of where they clicked.
How has this been tested?
Run locally via docker compose, open the locally hosted website in my browser.
Toggle the note layer via the layer panel, and the hide note layer button appears and disappears.
Click the add note button on the side, and the hide note layer button appears (as the note layer is activated).
Edit: Currently modifying the existing tests so they can handle the new functionality. I will remove the draft status when this is done.
Edit 2: Finally managed to get the tests to work correctly on my side. Removing the draft status.