Closed tony-o closed 10 years ago
@erickr thanks for taking a look! I'll take a pass at that on the next iteration, I'm working on integrating a wysiwyg type editor for editing notes. Great point on the notes and notebooks listing
@erickr took your recommendation and moved the notes to a left nav that collapses on a small screen. also made it so, while collapsed, swipe gestures can be used to show the notes.
Sounds great, will try it tomorrow night!
This looks great, but I do have some feedback I'd like you to take a look at before I merge. I also found a couple of deal-breaker bugs.
Holy crap! I only just now saw that you dumped in 25,000 + of JS files. I absolutely do not want these in the repo. They are not magpie code, and should not be treated as such. They are dependencies.
Currently, and external JS dependencies that are needed are just loaded through their various CDNs, which is probably a bad practice (magpie will look terrible if the user doesn't have an internet connection when they open it, if the static stuff isn't already cached.)
We should think about how to handle dependencies better. I think a good approach might be to check for their presence on launch, and download anything that's missing into the static dir. (We should add them to .gitignore if we do this, so devs don't accidentally put them into PRs.) There might be a way to do this as part of setup.py
. I'll look into that.
Can I also suggest use of git submodules? so that we can fix versions if any breaking upgrade happens ? And then we could use building job that gather the different dependencies and package everything?
What's your opinion?
@Erwyn suggestion taken and implemented in this last batch, thank you
@charlesthomas points 1, 2, and the extra libs being IN the repo has been pushed -
I tested this again, lots of my previous suggestions was now there, quickly done!
Perhaps we need to re-arrange the buttons a bit, but this seems really promising. Perhaps move the edit, star,delete buttons onto the breadcrumb-bar? There will be a history button as well soon hopefully for example, or we could perhaps replace it with a dropdown menu which could accomodate more options in the future.
Will be back with more comments, nice to see your work!
@tony-o This PR doesn't work. JS libs are missing - am I supposed to download them manually at this stage?
@charlesthomas did you run git submodule init and git submodule update ?
Instead of git submodules, you might want to look into using Bower as a front end asset manager. It functions similarly to submodules but is easier to manage and makes it much more clear and specific what scripts versions you are using.
Appears my email didn't make it into the thread.
@dangayle what does Bower provide that isn't supplied by git submodules? It looks like it just adds the requirement of NPM+Node.js. While I like JS/node, I fail to see the value that Bower would add here
Could we use pip and fanstatic to include the js-dependencies, that will avoid git submodules and also bower as dangayle suggested. Seems like it would be more pure python which might be more conservative on the requirements side. I havent tried it but read this answer, http://stackoverflow.com/a/19525324 which linked to http://www.fanstatic.org/en/latest/libraries.html
@erickr - it looks like Fantastic still keeps the JS as part of the code base which is what @charlesthomas had me remove initially, am I mis-skimming the Fantastic documentation?
@tony-o as I said I havent tried, but if that is the case I think it defeats the purpose of it, might have to test it to verify. I figured it should do something like phps composer and just install them on pip install into some vendor-folder or similar.
I am sensible at @dangayle argument.
I don't know what's the future of Magpie, but we can imagine that in a near future, we'll have a backend in python and a pure javascript frontend (maybe angularjs ?). If this is the way it is going to evolve then yes, bower is making more sense. At the moment, it is still fresh, but, maybe thinking about it?
If so, let's open a thread about it!
Just tested this, looks very good!
Although, imho, I think the list of notes should be in something like a left pane when using it on a desktop, for faster navigation between notes. On mobile the notes could perhaps be accessible by swiping horizontally to get to a list of notes in the selected notebook.
A bredcrumb might be good information as well, like / /
One issue I noticed was that if you have so many notes in a notebook that the dropdown is higher than the screen, then its a bit hard to take the ones at the bottom.