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

autoMarkRead scroll to mark as read #227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

primaeval
Copy link

Hi. This will let you scroll past items to mark them as read like Feedly does in title or full view.
I'm not a javascript expert so it probably needs some re-organisation.
I hope you will add it in as brief is just what I need for everyday rss reading.

@tanriol
Copy link
Member

tanriol commented Apr 2, 2017

This is an interesting idea, but not in the next version. I'm working on the internals right now and don't want to introduce behaviour changes.

@tanriol tanriol added this to the Future milestone Apr 2, 2017
@primaeval
Copy link
Author

primaeval commented Apr 2, 2017

Thanks. Do you think you can keep this addon going when XUL gets removed from Firefox?
I'm relying on it now for feeds that Feedly can't seem to keep up with, especially reddit.

@tanriol
Copy link
Member

tanriol commented Apr 2, 2017

Working on it. Brief is my primary feed reader, after all.

That's exactly why I'm removing XUL from Brief as much as possible. Unfortunately, there's some technical debt that has to be dealt with.

@primaeval
Copy link
Author

One last request for me while you're working on this project.
Could you make the titles wider in Headlines mode.
I changed it on one of my computers but not this one. It is a pain that Firefox doesn't let you install unsigned addons any more. I don't like trust running a dev version as my main browser.

@tanriol
Copy link
Member

tanriol commented Apr 2, 2017

You can do that yourself with a custom style (Options - Options - Customize) like

.headline-container {
  flex-basis: 1200px !important;
}

@primaeval
Copy link
Author

Thanks. That's handy. :)

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.

2 participants