search

Alex-Rose / fb-messenger-cli

fb-messenger-cli, console Facebook messenger
761 stars 99 forks source link

Add ability to see more than one screenful #212

Open jtgd opened 5 years ago

jtgd commented 5 years ago

It would be nice to scroll back in the conversation. Not sure how far you should allow, since I'm not sure what the API allows, but as much as feasible would be nice.

(BTW I LOVE this program! Thank you and I hope you keep it up to date. You should add it to Arch AUR)

p1ho commented 5 years ago

I'd say this is directly related to #175

jtgd commented 5 years ago

Yes, that's pretty much the same thing. Should I close this?

p1ho commented 5 years ago

I'd say if you want to make a pull request, you can keep both this and #175 open and you can reference both in the PR. If you do make a PR, I can review it, but I'm not a core maintainer so there's no guarantee on when the new changes can get merged.

ghost commented 5 years ago

I'd be interested in taking this on as an avid fan of the utility.

p1ho commented 5 years ago

@katipallya That sounds good! In case it helps, one of the challenges with history is how you can scroll up and down (like when you navigate up and down in $ git log). I looked at terminal-kit and node-pager, from my own playing-around I think node-pagermight be easier to get started, the example in its README works out of the box.

Edit: node-pager does not work for Powershell for me. I also looked at the source code, it literally just passes the string to a newly spawned child process, so that'll probably break things. Do you have a good package in mind?