Giter VIP home page Giter VIP logo

Comments (5)

v1r0x avatar v1r0x commented on June 10, 2024

I'm currently working on this one and the results looking good for

  • Context menu > Add new concept
  • Add broader concept
  • Add narrower concept

But what should happen after one deletes a concept using the options from the context menu?

  1. Delete concept and descendants
  2. Delete concept and move descendants one level up
  3. Delete concept and move descendants to the top level

My idea for 1. was to remove the concept completely, which would be easy to implement. But I couldn't think of a good behaviour for 2. and 3. Do you have any idea @dirkseidensticker @AngusLeBoeuf ?

from thesaurex.

dirkseidensticker avatar dirkseidensticker commented on June 10, 2024

As there are hypothetical use cases for this, I would suggest the following:

  • in any case there should be a warning, stating that deleting a concept that has multiple broader or narrower concepts results in the following ...
  • this warning must be accepted "'yes' i will delete them'
  • cases:
    1. is good as you proposed
    2. should only work on the selected element in the tree (the one you clicked)
    3. is not necessary and can be removed

from thesaurex.

v1r0x avatar v1r0x commented on June 10, 2024

I encountered several problems with our ideas.

One problem is your case 2 for an element as 'Textilverarbeitung'.

Example

  • Path 1: Fund > Artefakt > Form > Werkzeug > Textilverarbeitung
  • Path 2: Facetten > Artefakt (thematisch) > Werkzeug > Textilverarbeitung
    It has only one parent ('Werkzeug'), but 'Werkzeug' has two parents. Thus, moving the children of 'Textilverarbeitung' in Path 1 would move them in Path 2 as well.

Another problem is ('Textilverarbeitung' again) that the current implementation for an element with one parent, but two parents can not compute the exact path to this element in the tree.

We should discuss this in person next week.

from thesaurex.

v1r0x avatar v1r0x commented on June 10, 2024

Thanks to #63 I think this has been fixed. Can someone confirm this please? @eScienceCenter/spacialists

from thesaurex.

derntl avatar derntl commented on June 10, 2024

🎱

from thesaurex.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.