holodex / app

http://holodex.enspiral.com
GNU Affero General Public License v3.0
24 stars 1 forks source link

Legend widget #115

Closed simontegg closed 9 years ago

simontegg commented 9 years ago

Ok @ahdinosaur try this one out. (pr on data as well)

ahdinosaur commented 9 years ago

if we're using material-ui can we only require the parts that we actually use? it's a massive increase to our bundle size if we include everything. also btw it's not a dependency in the package.json yet.

simontegg commented 9 years ago

ok

ahdinosaur commented 9 years ago

is it on purpose that there's no legend to the 'subgroup', 'parent-group' relationship? that doesn't match what i expected.

also still can't visualize 'is-member-of' or 'has-member' relationships, as i'd expect, but that's the same as it is now.

simontegg commented 9 years ago

The legend-widget is set to only appears for context relationships. Though it could be done as you expect .My sense is that users primarily want to toggle the query on/off for non-context relationships.

ahdinosaur commented 9 years ago

maybe that's true about toggling, but there's nothing that describes what i'm seeing if i see orange arrows but no legend to describe what that means.

simontegg commented 9 years ago

That's right. The query-tag could have an associated link icon, or the legend widget could be present if links are shown, or the link description could appear on hover or click.

I don't have a high confidence in a particular solution to the above, so the thinking was was to solve one main problem (I want to toggle the links (and their targets) on/off while keeping the query present, where there is large distinction between toggled states.

ahdinosaur commented 9 years ago

yeah, i'm happy enough with this as is and keen to see how others react, just giving feedback on my reaction.

simontegg commented 9 years ago

fair enough

ahdinosaur commented 9 years ago

if https://github.com/holodex/app/commit/a1dc93af9b33ea3572b0e54a0c435e75a9919f38 works, then :+1: from me on merge.

ahdinosaur commented 9 years ago

sweet, i'll try and get this deployed tonight.