| Age | Commit message (Collapse) | Author |
|---|
|
|
|
|
|
|
|
|
|
|
|
|
|
One line, two changes:
keypress->keydown:
- fixes #137. Chrome doesn't dispatch a keypressed on tab. Firefox and
chrome both dispatch a keydown, so this works in both.
$(document)->element:
- fixes #171. Don't listen to text input events globally, only on the
element.
|
|
|
|
|
|
|
|
Also fix miscased scss filename.
|
|
|
|
|
|
|
|
|
|
|
|
See https://github.com/baconjs/bacon.js
Usage right now:
we have rz_bus to hold global busses:
- names: for autosuggest
- ui busses:
- ui_key
- ui_input
The events are objects with {where: from consts, keys/input}
(could maybe just use the events and rely on the target element
class/id)
And graph has it's own bus:
graph.diffBus
(camelcase probably not the right style - will fix in later commit)
To publish to a bus you use push.
To insert another stream to a bus you use plug.
To listen to a bus you use onValue (you can use subscribe too to get
errors and end of stream event, but we don't use that yet).
This allows usage of map, filter, flatMap, see github repo for cool
examples.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
After using up/down keys to select one of the entries you had an extra
enter required to "complete" the sentence.
Reproduce:
ctrl-alt-i (focus on 'Type here to create a network', #textanalysis)
The first enter is the selection, the second is ignored (the bug fixed
by this commit), the third is handled by the #textanalysis keypress
trigger.
The fix was simple, not sure why it works. Seems there is a
keyup/keypress interaction. The keyup was on the document element,
replaced by a keypress on the same element.
|
|
|
|
Possibly introduces required double enters sometimes. This is a result
of an exception thrown in the callback, needs work.
|
|
It now returns a node, either a new one or the existing one. This is
used by addNode and further up to get the existing (regressed until this
patch from the breaking patch) behavior of expanding existing nodes with
new links.
|
|
|
|
Note that despite the sad sad fact __defineGetter__ is not supported by
IE8, we are currently at quality 'Crap' on IE8 even without it (I
checked, courtesy 14 days trial of crossover from codeweavers), and Dor
says it is a level -1000 priority.
|
|
initialization
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
The added input lets you select by *oring* multiple space less
expressions. i.e. 'a b' will find all nodes with name containing either
'a' or 'b'. Not the specification, will be remedied later (i.e. maybe
tomorrow, who knows).
excuse the bulk commit, this does 3 things:
splits off selection logic into src/view/selection
- handles setting state explicityl on Node's, still bad.
- bad at least the 'selected' state (i.e. boolean) is in one place.
- common point for clicking on nodes/edges and selecting view selection
input
makes graph.getConnectedNodesAndLinks accept an array and not just a
single node
implements the search functionality outlined in the first paragraph.
|
|
|
|
Considered using jquery.hotkeys, even tried that, but didn't actually
work for me. This code seems to function under all browsers I tested (IE
11, chrome recent, FF recent - will test on IE 8).
|
|
|
|
|
|
Fix removed textanalysis autoSuggestName call using signal/slot, no
additional signal added (using graph_diff)
Broken in af20dd47d3cb254d4dd2638264fe79385a199832
|
|
|
|
|
