3DStreet / 3dstreet-editor

3DStreet Editor Repo
https://3dstreet.app
Other
18 stars 3 forks source link

feature/issue-408/entity-panel #412

Closed rostyslavvnahornyi closed 2 months ago

netlify[bot] commented 4 months ago

Deploy Preview for 3dstreet-editor-builds ready!

Name Link
Latest commit b2bba3608446e0299f0a5ad92112384a57b16286
Latest deploy log https://app.netlify.com/sites/3dstreet-editor-builds/deploys/662f0a0833f65a0008e3e8dd
Deploy Preview https://deploy-preview-412--3dstreet-editor-builds.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

kfarr commented 4 months ago
Algorush commented 3 months ago

I added new parameter mixinGroups to options array. For specifying one or few mixin group names to show if option is selected:

  const options = [
    {
      value: 'Layers: Streets & Intersections',
      label: 'Layers: Streets & Intersections',
      onClick: () => console.log('Layers: Streets & Intersections')
    },
    {
      value: 'Models: Personal Vehicles',
      label: 'Models: Personal Vehicles',
      mixinGroups: ['vehicles', 'vehicles-rigged']
    },
    {
      value: 'Models: Transit Vehicles',
      label: 'Models: Transit Vehicles',
      mixinGroups: ['vehicles-transit']
    },

I sugest to store cardsData with objects data (mixin id, img, name, description) in separate json file. And options object with mixin group and Layers data in separate json file too for convenient. Where is better to store these files? One question about options array. Now there are many more categories of mixins than group names in options. I have done it now so that all mixins are displayed, without dividing them into subgroups in selectedOption. I see few variants to do here:

Algorush commented 3 months ago

I added the ability to load layers by clicking on cards. And also svg-extruder element. All this is for testing. For now, all this is in one place - 'Layers: Streets & Intersections', it could be distributed among other groups later. All this works, but for example, when deleting a mapbox element from the scene, an error occurs in 3dstreet-editor, and also when deleting a new street, for some reason dummy elements are added to the structure of the existing street, after opening/closing the editor everything is fine. I will create isseus in Editor for it

kfarr commented 3 months ago

remaining before this can be merged:

Algorush commented 3 months ago

Now adding and previewing objects correspond to the elevation level of the segment for which they are added. And also in this PR, I made that objects are always added as direct descendants of the parent segment of the currently selected entity in the scene. In another branch, objects are always added to a special entity with class .custom-group

kfarr commented 2 months ago

Checklist to complete v1:

LATER:

kfarr commented 2 months ago

see #421