kormyen / memex Goto Github PK

Will be to do with the new loading setup in main.js

    var client = new XMLHttpRequest();
    client.open('GET', FILELOCATION);
    client.overrideMimeType("text/plain");
    client.onreadystatechange = function(req, res) 
    {
      if (client.responseText.trim() != '')
      {
        client.onreadystatechange = null;
        parent.setup(client.responseText);
      }
    }
    client.send();

Maybe 'onreadystatechange' is being called multiple times (on partial load(?), then on full load) but it currently only actions the first call.

Load divs on scroll?
Delayed div construction?

I believe the slow-ness/lag is due to constructing dom elements rather than dealing with the text data itself (so far...)

"add" seems broken, clicking on it removes the masonry layout but doesn't show the form input.

FYI you have a file linked on your memex 12018-07-15_GoodProgrammersKnowMoreAboutFeelingsThanComputers.pdf that doesn't exist.

Currently if the sidebar overflows, the overflow part gets hidden. On smaller screens this would keep user from clicking on some of the tags.

Adding overflow-y: scroll to .menu would make this section scrollable. This means having a scroll bar between sidebar and masonry view though - not very aesthetic.

On Chrome / other Chromium based browsers, we could use ::-webkit-scrollbar to style the scroll bar, and hide it with display: none. This won't work on Firefox (and probably Safari), but would work for the Electron build too.

If we want it to be scrollable and aesthetic - like Slack's sidebar scroller, for example - a 3rd party javascript library will have to be used :<

Here's the commit in my fork, for reference: dotcli@1020238

• Go back to prev query after ADD (don't clear)
• Proper quote formatting for ADD screen
• Clear add screen form input titles properly

Currently the tag filtering functionality breaks when there's a space inside the tag. Would be nice to support tags with multiple words - since some concepts are difficult to condense into one English word.

Our forks have diverged by 30 something commits at this point so I'm a bit afraid to do a pull request, but here's the fix in my fork. dotcli@808df0a .

I try call from electron index.html without success

Should the database be separated from this repository?
It works fine as is for me personally, but would be a hassle for other's pulling the Memex repo updates.

Git-sub-module like (I assume) @drisc 's https://github.com/drisc/mem ?
Or just entirely separate file like @joshavanier 's Log app.

memex displays a big error after following electron instructions

npm i
npm start

Uncaught TypeError: Cannot set property 'innerHTML' of null add.js:52

this.overlay isn't being set.

Options:

• Custom build faster view layout system
• Pagination (would need search system)
• Infinite scroll (would need search system)

Hi, thanks for building such a good tool! love the demo and the idea. But is there any doc about how to e.g., C(R)UD the snippet, how to edit tags etc? it'd be great without reading the code to get to use this thing! Appreciate it!

I had some (very basic) experience in Node but can't wrap up my mind. There's no way to do auth/login etc, i can see that. But i guess we can only edit the db file to add new knowledge snippet right?