cwrc / HuViz

LOD visualization tool for humanities datasets
8 stars 1 forks source link

with just a class engaged the instance names appear in current command #233

Closed smurp closed 5 years ago

smurp commented 5 years ago

The current command is contains an itemization of the selected set rather than a parsimonious expression containing just class names with exclusions.

Expected Behaviour

The selected set should be displayed in a fashion which reflects the user's intent as expressed through their clicks. An itemization of the selected set obscures the fact that the user merely clicked just a class in the picker, not a bunch of instances.

Current Behaviour

update_english() is somehow generating a non-parsimonious expression.

Steps to Reproduce (for bugs)

  1. http://localhost:9997/#load+https://raw.githubusercontent.com/cwrc/testData/master/demoData/WoolfsOrlando.ttl+with+http://sparql.cwrc.ca/ontology/cwrc.ttl

  2. click sexuality in class picker (because it has just two instances)

  3. move mouse out of the class picker

  4. note that the current command now reads ___ bisexual an lesbian . instead of the shorter ___ Sexuality . which correctly conveys the users intention.

smurp commented 5 years ago

This is HIGHEST PRIORITY (with a bullet) because it is making alpha unusable because more populous classes (such as Thing) make the UX very twitchy and kafka-esque.

smurp commented 5 years ago

closed by https://github.com/smurp/huviz/commit/38ae8c7b992efabb73eebfe65d83b9d70e5e5afa

accuracy of Current Command remains an issue with mixed classes and instances engaged. See #36