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

Prevent showing commit dialog when starting to edit a layer that is not in relation with last edited layer #115

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

volterra79
Copy link
Member

@volterra79 volterra79 commented Jun 26, 2024

Backports: #116 to v3.7.x

…an doesn't in relation of last toolbox selected
@volterra79 volterra79 added the bug Something isn't working label Jun 26, 2024
@volterra79 volterra79 added this to the 3.7 milestone Jun 26, 2024
@volterra79 volterra79 self-assigned this Jun 26, 2024
@volterra79 volterra79 marked this pull request as draft June 26, 2024 10:12
volterra79 added a commit to g3w-suite/g3w-admin that referenced this pull request Jun 26, 2024
volterra79 added a commit that referenced this pull request Jun 26, 2024
@Raruto Raruto changed the title Avoid to ask commit changes of a layer when start to edit a layer that is not in relation with last edited layer Prevent showing commit dialog when starting to edit a layer that is not in relation with last edited layer Jul 2, 2024
wlorenzetti added a commit to g3w-suite/g3w-admin that referenced this pull request Jul 3, 2024
@volterra79 volterra79 marked this pull request as ready for review July 4, 2024 15:24
@volterra79 volterra79 merged commit 5472d54 into v3.7.x Jul 5, 2024
@volterra79 volterra79 deleted the issue_114 branch July 5, 2024 06:32
volterra79 added a commit to g3w-suite/g3w-admin that referenced this pull request Jul 11, 2024
volterra79 added a commit to g3w-suite/g3w-admin that referenced this pull request Jul 12, 2024
@volterra79 volterra79 mentioned this pull request Aug 6, 2024
volterra79 added a commit that referenced this pull request Aug 6, 2024
…ot in relation with last edited layer (#116)

* 🐛 Port #115 to v3.8.x

* code format

* move `toolboxidactivetool` into `Editing.vue`

* 🐛 set features to [] in case of no features get

* promisify + jsdoc for commit dirty

* i18n + convert to async funcion: `toolbox.revert()`

* attempt to fix revert

* remove doubtful function: `revert()`

* `indexOf` → `includes`

* new constructor param: `ToolBox(layer, dependencies)`

* 🐛 Commit dirty layer features

* ✨ Zoom to constraint scale on map click

* show `ol-zoom-in` cursor when `false == canEdit`

* 🎨watch select and canEdit in one watch

* add helper function: `getActiveTool`

* 🐛 Use one watch only for check changes of control, selection and edit

* console.log

* getInteraction()

* click on panel to fit zoom constraint

* simplify watcher

* prevent "clicktozoom" behavior on "start-editing" button

* 🔥 Remove unused CopyFeaturesFromOtherProjectLayer.vue

* missing cursor

* ♻️ Move cursor scale constraint to _handleScaleConstraint method. Need to remove this.unwatches ?

* Use setMouseCursor

* remove console.log

* fix zoom-in cursor

* extract const

* when stop toolbox, in case of scale constraint need to remove event handler on change:resolution

* Fix show zoom in when not in editing

* Fix show zoom on start

* 3.9.0-alpha.0

* Change version to v3.9.0-alpha.0

---------

Co-authored-by: Raruto <[email protected]>
volterra79 added a commit that referenced this pull request Aug 6, 2024
* Clean code - spaces

* Clean code - spaces

* Clean code - spaces and comments

* Clean code - spaces and comments

* Clean code - spaces

* Clean code - spaces

* Clean code - spaces

* Update version info

* Clean code - spaces

* 🐛 Check if there is an active tool on toolbox

* 🐛 Port #115 to v3.8.x

* 🐛 set features to [] in case of no features get

* 3.8.1

* Revert "3.8.1"

This reverts commit 8859cb6.

* 3.8.1-alpha.0

---------

Co-authored-by: Raruto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant