Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional features, more user-friendly UI #120

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JCulley
Copy link

@JCulley JCulley commented Aug 7, 2018

Having modified the application adding features I feel needed to be improved upon I figured I'd share them.

File based support
The one main issue I faced was working on AIX unfortunately Redis is not supported so I modified the application to utilize NodeJS and created a file system which writes room data in JSON to individual files.

Moved and updated toolbar
Using Scrumblr one issue faced was with tall boards the toolbar would be out of view making it difficult to add new cards, which is why the new design has a static toolbar which follows the page on the right hand side.

Additional card size
A very large card option was also added in case you're looking to write longer notes.

Font options
This updated version also includes new toolbar options to cycle through various fonts, and additionally change the font size on the cards.

Password protect
If your implementation is running in the public domain you may not want everyone to be able to view your board depending on how confidential the information is. A toolbar option was added allowing you to lock the board with a password so if the page is visited it must be entered otherwise the board will not load.

@ocdtrekkie
Copy link

Thanks for sharing! It's amazing to me so many folks are still hacking on and adding to this codebase. I would say it would've been "more ideal" if the commits were more granular: I maintain a Scrumblr fork, and I might be interested in some of these changes but not all of them, and having all of these varied changes in a single commit makes it hard to tell what is what.

@JCulley
Copy link
Author

JCulley commented Aug 10, 2018

Yea it would have been better if I had the intentions of publicly releasing my changes from the beginning however as we're using it internally within the team I work with they originally wanted to keep the revisions private for our own use.

@aliasaria
Copy link
Owner

Getting this error when I try to test:

/Users/ali/workspace/scrumblr/server.js:365
db.getFont( room, function(font) {
^

TypeError: db.getFont is not a function
at /Users/ali/workspace/scrumblr/server.js:365:6
at getRoom (/Users/ali/workspace/scrumblr/server.js:522:2)
at initClient (/Users/ali/workspace/scrumblr/server.js:362:2)
at Socket. (/Users/ali/workspace/scrumblr/server.js:120:5)
at Socket.emit (events.js:315:20)
at /Users/ali/workspace/scrumblr/node_modules/socket.io/lib/socket.js:503:12
at processTicksAndRejections (internal/process/task_queues.js:79:11)

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.

None yet

3 participants