Skip to content

Doubly-linked Items#9

Merged
raineorshine merged 44 commits into
masterfrom
doubly-linked
Jul 29, 2019
Merged

Doubly-linked Items#9
raineorshine merged 44 commits into
masterfrom
doubly-linked

Conversation

@raineorshine
Copy link
Copy Markdown
Contributor

@raineorshine raineorshine commented Jul 14, 2019

Store children in parents for faster children lookup.

Parents are still stored in children for performant parents lookup. Both must be maintained in parallel.

@raineorshine raineorshine changed the title Double-linking: Store children in parents for faster children lookup. Double-linked Items Jul 14, 2019
@raineorshine raineorshine changed the title Double-linked Items Doubly-linked Items Jul 14, 2019
Allow the results of the new getChildrenWithRank which uses contextChildren to be compared against getChildrenWithRankDEPRECATED which uses inefficient memberOf collation to test for functional parity at the given probability between 0 (no testing) and 1 (test every call to getChildrenWithRank)
Both removing one thought of many (in which the entry will still have an array) and removing the last thought (entry removed from contextChildren).
@raineorshine raineorshine merged commit f065f9f into master Jul 29, 2019
@raineorshine raineorshine deleted the doubly-linked branch July 29, 2019 07:38
cindmichelle added a commit to cindmichelle/em that referenced this pull request May 11, 2026
…review)

Addresses review items cybersemics#2cybersemics#9:
- Bump "Gestures" header to semibold (cybersemics#2)
- Set command subtitle to regular weight (cybersemics#3)
- Reduce gesture diagram from 32 to 26 (cybersemics#4)
- Tighten header→divider gap (cybersemics#5)
- Tighter non-highlighted row spacing, larger gap below highlighted row (cybersemics#6, cybersemics#7)
- Shrink Cancel-block gap; flush to top when main list is empty (cybersemics#8, cybersemics#9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant